rpuch commented on code in PR #4361:
URL: https://github.com/apache/ignite-3/pull/4361#discussion_r1756240161
##########
modules/metastorage/src/main/java/org/apache/ignite/internal/metastorage/impl/MetaStorageLeaderElectionListener.java:
##########
@@ -179,6 +197,12 @@ private interface Action {
* Executes the given action if the current node is the Meta Storage
leader.
*/
private void execute(Action action) {
+ if (leaderSecondaryDutiesPaused.getAsBoolean()) {
Review Comment:
We don't need protection provided by the busy lock to check whether
secondary duties are paused; and without checking the lock, we can leave the
method faster if they are paused. So I don't see why we would need to do it
under the lock.
--
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]
