DeathGun44 commented on PR #5429: URL: https://github.com/apache/fineract/pull/5429#issuecomment-3829291795
@IOhacker After extensive local testing with 400+ concurrent deposits, I was unable to reproduce the 320 SimpleAsyncTaskExecutor threads described in the ticket. The notification async path (via applicationEventMulticaster) doesn't trigger in my local environment despite having users with ALL_FUNCTIONS permission in the same office as the savings account. The fix remains architecturally correct (replacing unbounded SimpleAsyncTaskExecutor with bounded ThreadPoolTaskExecutor), but without production-like data I cannot demonstrate the specific issue from FINERACT-1934. Given reproduction requires production-scale data, I'll close this PR for now. Happy to revisit if someone can provide a reproducible test case. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
