Lea Fan created FINERACT-2787:
---------------------------------
Summary: Ineffective method-local synchronization in GSIM
application submission
Key: FINERACT-2787
URL: https://issues.apache.org/jira/browse/FINERACT-2787
Project: Apache Fineract
Issue Type: Improvement
Components: Savings
Reporter: Lea Fan
Background
The submitGSIMApplication method submits savings applications for the clients
belonging to a GSIM group.
Current implementation
The method creates a new lock object for each invocation:
final Object lock = new Object();
synchronized (lock) {
// submitApplication(...)
}
Problem
Because the lock object is created inside the method, every invocation receives
a different lock instance.
Therefore, two concurrent requests executing submitGSIMApplication do not
synchronize with each other. Each request locks only its own private object.
Within a single invocation, the existing for-loop already invokes
submitApplication sequentially, so the synchronized block does not provide
additional ordering for that loop.
This may give readers the misleading impression that concurrent GSIM
application submissions are coordinated, although the lock cannot provide
cross-request mutual exclusion.
This issue does not claim that a user-visible concurrency failure has already
been reproduced. The intended concurrency requirement should first be clarified.
Expected behavior
Clarify the intended concurrency semantics of GSIM application submission:
* If cross-request mutual exclusion is not required, remove the ineffective
local lock.
* If cross-request coordination is required, use an appropriate shared,
transactional, or database-level mechanism based on the business invariant
being protected.
Acceptance criteria
* The method-local lock is removed or replaced with a mechanism that actually
implements the intended concurrency behavior.
* Tests cover the intended GSIM application submission behavior.
* The implementation does not imply concurrency protection that it cannot
provide.
Related issues
* FINERACT-603 - Original GSIM and GLIM implementation
* FINERACT-2237 - GSIM parent account creation fix
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
