tarpas / pytest-testmon

Selects tests affected by changed files. Executes the right tests first. Continuous test runner when used with pytest-watch.
https://testmon.org
MIT License
800 stars 54 forks source link

fix: Increase sqlite connect timeout to 60s #220

Closed smitp closed 12 months ago

smitp commented 1 year ago

Related issue

sqlite3.connect

Default timeout 5s.

Change sqlite connect timeout to 60s to avoid database locking issues. This works wit h -n16 xdist workers.

smitp commented 12 months ago

Hi @tarpas Just checking in on the status of this PR. If you need any more information or changes, please let me know. Looking forward to your feedback!

tarpas commented 12 months ago

I feel like this is sweeping a problem under the rug. By design there should be no contention for writes.

I asked questions in the original issues and we also made a lot of effort to reproduce it. Some questions were not answered. I'm going to reiterate in the issue.