Code monitors: fix issues with concurrent access to transactions
There are no commits yet
Push commits to the source branch or add previously merged commits to review them.
Created by: camdencheek
This is a super hacky fix for the fact that creating a code monitor executes in a transaction which must not be used concurrently.
Honestly, this is a really scary problem across the whole codebase. We have no idea whether the database handles we are using are actually transactions, so we have no idea whether we can use them concurrently.
I plan to spend some time immediately on fixing this problem because I expect I'm not the only one who hits this.
Manually tested that creating a code monitor for a bunch of repos succeeds.
Push commits to the source branch or add previously merged commits to review them.