You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@drill.apache.org by Ben-Zvi <gi...@git.apache.org> on 2017/11/03 22:10:11 UTC

[GitHub] drill pull request #:

Github user Ben-Zvi commented on the pull request:

    https://github.com/apache/drill/commit/a9f2a1c6ad59386828f41731b7415e18a9459cc6#commitcomment-25390233
  
    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 188:
    Checking on https://docs.oracle.com/javase/8/docs/api/java/util/concurrent/locks/Condition.html#await-- , looks like there is no need to re-grab the lock after calling await() . Also all the examples I found online don't do so.  It is possible that calling lock() again actually increments the (already acquired) lock counter.


---