Github user arina-ielchiieva commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1051#discussion_r154679303
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/rm/ThrottledResourceManager.java
 ---
    @@ -288,8 +287,10 @@ public void setCost(double cost) {
         }
     
         @Override
    -    public void admit() throws QueueTimeoutException, QueryQueueException {
    -      lease = rm.queue().enqueue(foreman.getQueryId(), queryCost);
    +    public void admit() {
    +      QueueAcquirer queueAcquirer = new QueueAcquirer(rm.queue(), foreman, 
queryCost);
    +      queueAcquirerThread = new Thread(queueAcquirer);
    --- End diff --
    
    Admit method is moved back to be blocking.


---

Reply via email to