This repository has been archived by the owner on Aug 28, 2020. It is now read-only.
Bug: Concurrency is broken in Schedule #922
Labels
Mod: SettingsGateway
Issues and PRs related to SettingsGateway.
Priority: High
Issues that must be fixed or PRs that must be finished and merged with high priority.
Describe the issue
If you have 10 tasks and they all end at nearly the same time, chances are 100% that they all will read the state before being updated by the other writes, meaning that the last to write to DB will restore the other 9. Thanks @gc for noticing this bug.
Code or steps to reproduce
Expected and actual behavior
It should do either sequentially or in a burst (using
@klasa/request-handler
to do an individual update then one update containing 9 changes).Further details
fe7df708e44e0280dfaf0f8e457b154781bb5140
42fdd6d4b8ec1596e26f21e03a415de179b816ba
The text was updated successfully, but these errors were encountered: