gianm commented on code in PR #12901:
URL: https://github.com/apache/druid/pull/12901#discussion_r945351914
##########
indexing-service/src/main/java/org/apache/druid/indexing/overlord/TaskQueue.java:
##########
@@ -82,14 +84,21 @@
*/
public class TaskQueue
{
- private final long MANAGEMENT_WAIT_TIMEOUT_NANOS =
TimeUnit.SECONDS.toNanos(60);
- private final long MIN_WAIT_TIME_MS = 100;
+ private static final long MANAGEMENT_WAIT_TIMEOUT_NANOS =
TimeUnit.SECONDS.toNanos(60);
+ private static final long MIN_WAIT_TIME_MS = 100;
+ // Task ID -> Task that is currently running
Review Comment:
I changed the comment to hopefully be clearer. It now reads:
```
// Task ID -> Task, for tasks that are active in some way (submitted,
running, or finished and to-be-cleaned-up).
```
--
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]
