Github user sohami commented on a diff in the pull request:
https://github.com/apache/drill/pull/633#discussion_r85557510
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java ---
@@ -295,7 +295,7 @@ public FragmentExecutor getFragmentRunner(final
FragmentHandle handle) {
*/
private class StatusThread extends Thread {
public StatusThread() {
- setDaemon(true);
+ setDaemon(false);
--- End diff --
Not sure why removing it will not ensure the same. I think it's redundant
and like overriding a default already set in constructor with default value
using setter method.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
