4 ms·
Actually, they added a unit test in a follow-on change: https://www.sqlite.org/src/info/dc7dd2d3e50e7cc4 https://www.sqlite.org/src/info/dc7dd2d3e50e7cc4
by otoolep 4y ago
Actually, they added a unit test in a follow-on change:
https://www.sqlite.org/src/info/dc7dd2d3e50e7cc4 https://www.sqlite.org/src/info/dc7dd2d3e50e7cc4
- zoomablemind 4y agoIndeed, thanks for pointing this one out. The test seems to test a shared access in rather a serial order. I wonder if underneath this is actually running as concurrent processes?
- otoolep 4y agoI may not be following your point, but I don't believe the root cause of this issue was a race condition, or anything that might be related to concurrency. The fundamental cause AFAIK was a SQLite connection was attempting to make a state transition (from one type of locking state to another) which shouldn't be allowed under certain circumstances, but the implementation didn't actually enforce this rule. So the added test really does test the root cause.