Github user vrozov commented on a diff in the pull request: https://github.com/apache/drill/pull/1196#discussion_r178375583 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java --- @@ -86,6 +86,9 @@ private final StatusThread statusThread; private final Lock isEmptyLock = new ReentrantLock(); private final Condition isEmptyCondition = isEmptyLock.newCondition(); + private boolean isShutdownTriggered = false; --- End diff -- Is this boolean necessary? Can you delay getting `isEmptyCondition` till shutdown is requested and use it in place of `isShutdownTriggered`?
---