Harsh Mehta
Harsh Mehta
@justinmclean Good catch — you’re right that the current implementation sets the flag before running runnable.run(), which means it could remain true even if the load fails. It also changes...
@justinmclean, I observed the following issues: - The method is called from `JcasbinAuthorizer.loadRolePrivilege()`. - The runnable modifies shared state (`allowEnforcer`, `denyEnforcer`). - It performs I/O operations (database reads). - Multiple...
> Yes, that looks much better. @justinmclean Thank you for your feedback. I’ve updated the PR with the requested changes. Please review it at your convenience.
@justinmclean Can I pick this?
@Shaddoll I am new to contributing to open-source projects and would love to get involved in this. I have a question regarding the implementation of the `ActiveClusterSelectionPolicy` and `CronOverlapPolicy`. Could...
Then I believe there’s no need to add these two parameters at the moment, right?
You can go ahead and assign this issue to someone else since I’m still new to contributing. Please let me know if there are any other issues that I might...
@kavirajk I'm new to open source and would love to contribute. Can you provide more details on how to implement this feature this? Any guidance on how to get started...
@kwin I got your point. ## Solution A new phase `ScmCheckRemoteSyncPhase` has been added to the beginning of the prepare phase sequence (right after `check-poms`). This phase: 1. **Checks if...