airajena commented on PR #5566: URL: https://github.com/apache/fineract/pull/5566#issuecomment-4013201921
> Woah, fascinating. Would we then be using `innodb_snapshot_isolation=OFF` during tests and also recommending it for prod/deployments as well? If this is a change we'll be making everywhere, please write the dev list and see if anyone has input/feedback/advice. I don't have experience with this database setting. Actually for this PR, I treated it as a compatibility setting for the MariaDB 12 upgrade path, not as a claim that Fineract should globally prefer. I agree this deserves wider input if we intend to keep applying it outside CI/tests, I will explain this in detail in mailing list, let's see if we get a better way. There are really two possible directions. 1. Keep innodb_snapshot_isolation=OFF for MariaDB 12 so runtime behavior stays aligned with what Fineract has already been validated against. 2. Keep MariaDB 12 defaults and revisit the affected Fineract transaction/locking flows so they are explicitly correct under the new upgrade. -- 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]
