-
Notifications
You must be signed in to change notification settings - Fork 531
Description
ACA-Py is used in highly scaled deployments, including multiple instances running across multiple nodes of a cluster. While to this point we have not seen a need for the implementation of a fully distributed lock to handle a transactional update to data across the entire cluster, it is quite possible that we do or will need such a capability. The issue #3738 is an example of that -- the need for an issuer to apply a unique number from a sequence to a credential across the entire deployment. While a solution for that specific issue was found (a database transaction that had been used previously had been inadvertently dropped in a refactoring), it did raise the question about whether a general purpose distributed lock should be added to ACA-Py.
If this is to be considered, please look at the comments in #3738 about approaches for such an implementation. As part of the work done for that Issue, PR #3782 was created towards that goal. Worked stop when the problem with the refactoring was discovered. That PR could be continued if it is decided that a distributed lock would be helpful.