cryptoe commented on code in PR #13288:
URL: https://github.com/apache/druid/pull/13288#discussion_r1008870427
##########
extensions-core/multi-stage-query/src/main/java/org/apache/druid/msq/indexing/MSQWorkerTaskLauncher.java:
##########
@@ -472,6 +471,32 @@ private void sleep(final long sleepMillis, final boolean
shuttingDown) throws In
}
}
+ /**
+ * Information about current status of running tasks
+ */
+ public static class WorkerStatus
Review Comment:
Let's declare this class in another class file since it's used in controller
impl.
Also, should we just change the name to WorkerCounts and rename the method
accordingly?
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
