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.
---
|