leventov commented on a change in pull request #7088: Improve parallelism of
zookeeper based segment change processing
URL: https://github.com/apache/incubator-druid/pull/7088#discussion_r279076392
##########
File path:
server/src/main/java/org/apache/druid/server/coordinator/CuratorLoadQueuePeon.java
##########
@@ -59,40 +68,50 @@
private static final int DROP = 0;
private static final int LOAD = 1;
- private static void executeCallbacks(List<LoadPeonCallback> callbacks)
- {
- for (LoadPeonCallback callback : callbacks) {
- if (callback != null) {
- callback.execute();
- }
- }
- }
-
private final CuratorFramework curator;
private final String basePath;
private final ObjectMapper jsonMapper;
private final ScheduledExecutorService processingExecutor;
+ /**
+ * Threadpool with daemon threads running scheduled tasks that monitor
whether
+ * the zk nodes created for segment processing are removed
+ */
+ private final ScheduledExecutorService monitorNodeRemovedExecutor;
+
+ /**
+ * Threadpool with daemon threads that execute callback actions associated
+ * with loading or dropping segments.
+ */
private final ExecutorService callBackExecutor;
private final DruidCoordinatorConfig config;
private final AtomicLong queuedSize = new AtomicLong(0);
private final AtomicInteger failedAssignCount = new AtomicInteger(0);
+ /**
+ * Needs to be thread safe since it can be concurrently accessed via
+ * loadSegment(), actionCompleted(), getSegmentsToLoad() and stop()
Review comment:
Please make method refs Javadoc links
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
