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`?
---
