Murtadha Hubail has submitted this change and it was merged. Change subject: [NO ISSUE][FAIL] Use Formatted Exception For Node Not Live ......................................................................
[NO ISSUE][FAIL] Use Formatted Exception For Node Not Live - user model changes: no - storage format changes: no - interface changes: no Change-Id: I06c42a8205c439fd2d320f053c79ed803aaa6b4a Reviewed-on: https://asterix-gerrit.ics.uci.edu/2861 Reviewed-by: Michael Blow <[email protected]> Sonar-Qube: Jenkins <[email protected]> Tested-by: Jenkins <[email protected]> Contrib: Jenkins <[email protected]> Integration-Tests: Jenkins <[email protected]> --- M hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/executor/JobExecutor.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Anon. E. Moose #1000171: Jenkins: Verified; No violations found; ; Verified Michael Blow: Looks good to me, approved diff --git a/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/executor/JobExecutor.java b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/executor/JobExecutor.java index dee36e9..bdc73d5 100644 --- a/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/executor/JobExecutor.java +++ b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/executor/JobExecutor.java @@ -470,7 +470,7 @@ throw new HyracksException("No satisfiable location found for " + taskAttempt.getTaskAttemptId()); } if (!liveNodes.contains(nodeId)) { - throw new HyracksException("Node " + nodeId + " not live"); + throw HyracksException.create(ErrorCode.NO_SUCH_NODE, nodeId); } return nodeId; } -- To view, visit https://asterix-gerrit.ics.uci.edu/2861 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: merged Gerrit-Change-Id: I06c42a8205c439fd2d320f053c79ed803aaa6b4a Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha Hubail <[email protected]> Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Michael Blow <[email protected]> Gerrit-Reviewer: Murtadha Hubail <[email protected]> Gerrit-Reviewer: Till Westmann <[email protected]>
