-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36168/
-----------------------------------------------------------
Review request for drill, Jacques Nadeau and Sudheesh Katkam.
Bugs: DRILL-3167
https://issues.apache.org/jira/browse/DRILL-3167
Repository: drill-git
Description
-------
when a query fails Foreman moves to a FAILING state until all fragment return a
terminal state. The Web UI will still display FAILED instead of FAILING
TestDrillbitResilience#failsWhenSendingFragments exposes a limitation to this
approach: if an error occurs when setting up remote fragments we can't assume
they will be able to return a terminal state. In this case the Foreman will not
wait for them to finish and return FAILED to the client immediately.
Diffs
-----
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/ProfileResources.java
6656bf6
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/ProfileWrapper.java
dd26a76
exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/Foreman.java
716fb66
exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/QueryManager.java
9318233
exec/java-exec/src/main/resources/rest/profile/list.ftl cf92ede
exec/java-exec/src/main/resources/rest/profile/profile.ftl 46cdc83
exec/java-exec/src/test/java/org/apache/drill/exec/server/TestDrillbitResilience.java
ce09f68
protocol/src/main/java/org/apache/drill/exec/proto/UserBitShared.java e76d748
protocol/src/main/java/org/apache/drill/exec/proto/beans/QueryResult.java
474e330
protocol/src/main/protobuf/UserBitShared.proto 0451fd2
Diff: https://reviews.apache.org/r/36168/diff/
Testing
-------
unit tests are passing, ongoing cluster testing...
Thanks,
abdelhakim deneche
