[GitHub] [druid] ArvinZheng commented on a change in pull request #10280: recreate the balancer executor only when needed

[GitBox <gi...@apache.org>]

ArvinZheng commented on a change in pull request #10280:
URL: https://github.com/apache/druid/pull/10280#discussion_r485913685



##########
File path: server/src/main/java/org/apache/druid/server/coordinator/DruidCoordinator.java
##########
@@ -653,8 +683,8 @@ private void stopBeingLeader()
     private final long startTimeNanos = System.nanoTime();
     private final List<CoordinatorDuty> duties;
     private final int startingLeaderCounter;
-    private int cachedBalancerThreadNumber;
-    private ListeningExecutorService balancerExec;
+    //private int cachedBalancerThreadNumber;

Review comment:
       oh, good catch, will remove them, thanks for reviewing




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org