Github user ppadma commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1051#discussion_r153293973
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/Foreman.java ---
    @@ -177,9 +155,10 @@ public Foreman(final WorkerBee bee, final 
DrillbitContext drillbitContext,
         queryManager = new QueryManager(queryId, queryRequest, 
drillbitContext.getStoreProvider(),
             drillbitContext.getClusterCoordinator(), this);
     
    -    recordNewState(QueryState.ENQUEUED);
    -    enqueuedQueries.inc();
    +    recordNewState(QueryState.PREPARING);
    --- End diff --
    
    Better to handle all state transitions under moveToState.


---

Reply via email to