Re: Review Request 36208: DRILL-3455: If a drillbit, that contains fragments for the current query, dies the QueryManager will fail the query even if those fragments already finished successfully

Wed, 22 Jul 2015 17:18:37 -0700 

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36208/
-----------------------------------------------------------

(Updated July 23, 2015, 12:18 a.m.)


Review request for drill, abdelhakim deneche and Jacques Nadeau.


Changes
-------

Addressed Jacques' comments


Bugs: DRILL-3455
    https://issues.apache.org/jira/browse/DRILL-3455


Repository: drill-git


Description
-------

DRILL-3455: If fragments on unregistered Drillbits finished successfully, do 
not fail the query

+ DRILL-3448: Flipped the atLeastOneFailure condition in QueryManager
+ fixes in DrillbitStatusListener interface
+ logs from implementations of DrillbitStatusListener


Diffs (updated)
-----

  
exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/DrillbitStatusListener.java
 80f2ca1 
  
exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/QueryManager.java
 9318233 

Diff: https://reviews.apache.org/r/36208/diff/


Testing
-------

passes unit tests


Thanks,

Sudheesh Katkam

Reply via email to