[GitHub] drill pull request #1237: DRILL-6348: Fixed code so that Unordered Receiver ...

[vrozov]

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

    https://github.com/apache/drill/pull/1237#discussion_r184585775
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/unorderedreceiver/UnorderedReceiverBatch.java
 ---
    @@ -149,25 +149,32 @@ private RawFragmentBatch getNextBatch() throws 
IOException {
         }
       }
     
    +  private RawFragmentBatch getNextNotEmptyBatch() throws IOException {
    +    RawFragmentBatch batch;
    +    try {
    +      stats.startWait();
    --- End diff --
    
    Yes, it does matter. If something goes wrong during `startWait()` there 
should be no call to `stopWait()`.


---