Github user ilooner commented on the pull request:

    
https://github.com/apache/drill/commit/a9f2a1c6ad59386828f41731b7415e18a9459cc6#commitcomment-25391123
  
    In exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java:
    In exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java 
on line 192:
    I wanted the method to wait for all the fragments and queries to complete 
even if it was interrupted. I don't think there is any benefit to exiting 
prematurely during the shutdown process because of an interrupt. I think it's 
better to ignore the interrupt and continue waiting for the exit condition. One 
possible reason for obeying the interrupt would be to prevent the system from 
hanging during the shutdown of a corrupted Drillbit. But the waitToExit method 
is already time bound to 30 seconds, so we will not hang.


---

Reply via email to