Murtadha Hubail has submitted this change and it was merged. Change subject: [ASTERIXDB-2042][CLUS] Handle Reporting Startup Completion Failure ......................................................................
[ASTERIXDB-2042][CLUS] Handle Reporting Startup Completion Failure - user model changes: no - storage format changes: no - interface changes: no Details: - if the NC fails to report its startup completion message, to the CC, shut it down so that it can be restarted again when the CC is back up. Change-Id: Ia69efb1538e57242a0a69ac1a0158a5be06ab866 Reviewed-on: https://asterix-gerrit.ics.uci.edu/1945 Sonar-Qube: Jenkins <[email protected]> Integration-Tests: Jenkins <[email protected]> Tested-by: Jenkins <[email protected]> Reviewed-by: Till Westmann <[email protected]> Reviewed-by: abdullah alamoudi <[email protected]> --- M asterixdb/asterix-app/src/main/java/org/apache/asterix/app/replication/message/StartupTaskResponseMessage.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Anon. E. Moose #1000171: abdullah alamoudi: Looks good to me, approved Till Westmann: Looks good to me, approved Jenkins: Verified; No violations found; Verified Objections: Jenkins: diff --git a/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/replication/message/StartupTaskResponseMessage.java b/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/replication/message/StartupTaskResponseMessage.java index 1611507..9125857 100644 --- a/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/replication/message/StartupTaskResponseMessage.java +++ b/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/replication/message/StartupTaskResponseMessage.java @@ -63,6 +63,7 @@ try { broker.sendMessageToCC(result); } catch (Exception e) { + success = false; LOGGER.log(Level.SEVERE, "Failed sending message to cc", e); } } finally { -- To view, visit https://asterix-gerrit.ics.uci.edu/1945 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia69efb1538e57242a0a69ac1a0158a5be06ab866 Gerrit-PatchSet: 2 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: Murtadha Hubail <[email protected]> Gerrit-Reviewer: Till Westmann <[email protected]> Gerrit-Reviewer: abdullah alamoudi <[email protected]>
