strichliste / strichliste-backend

Strichliste Backend
https://www.strichliste.org
Other
24 stars 14 forks source link

Refactor transaction handling to allow sqlite again #54

Open schinken opened 5 years ago

schinken commented 5 years ago

I tried changing code to update atomically. But then I can't correctly check the user account boundaries. Because there might be a race condition between simultaneous update requests. dang.

Why can't sqlite just support PESSIMISTIC WRITE :( https://github.com/strichliste/strichliste-backend/blob/master/src/Service/TransactionService.php#L81