Michael Blow has submitted this change and it was merged. Change subject: [NO ISSUE] Address comments in ReconnectingIPCHandle ......................................................................
[NO ISSUE] Address comments in ReconnectingIPCHandle Change-Id: I916939f3419cdc6983253b4f6d8b46756fa13e69 Reviewed-on: https://asterix-gerrit.ics.uci.edu/2287 Reviewed-by: Murtadha Hubail <[email protected]> Sonar-Qube: Jenkins <[email protected]> Integration-Tests: Murtadha Hubail <[email protected]> Tested-by: Murtadha Hubail <[email protected]> --- M hyracks-fullstack/hyracks/hyracks-ipc/src/main/java/org/apache/hyracks/ipc/impl/ReconnectingIPCHandle.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Jenkins: No violations found Murtadha Hubail: Looks good to me, approved; Verified; Verified diff --git a/hyracks-fullstack/hyracks/hyracks-ipc/src/main/java/org/apache/hyracks/ipc/impl/ReconnectingIPCHandle.java b/hyracks-fullstack/hyracks/hyracks-ipc/src/main/java/org/apache/hyracks/ipc/impl/ReconnectingIPCHandle.java index f0da860..9a2fc80 100644 --- a/hyracks-fullstack/hyracks/hyracks-ipc/src/main/java/org/apache/hyracks/ipc/impl/ReconnectingIPCHandle.java +++ b/hyracks-fullstack/hyracks/hyracks-ipc/src/main/java/org/apache/hyracks/ipc/impl/ReconnectingIPCHandle.java @@ -72,11 +72,11 @@ if (delegate.isConnected()) { return delegate; } + LOGGER.warn("ipcHandle {} disconnected; will attempt to reconnect {} times", delegate, reconnectAttempts); listener.ipcHandleDisconnected(delegate); delegate = ipc.getHandle(getRemoteAddress(), reconnectAttempts); - LOGGER.warn("ipcHandle " + delegate + " restored"); + LOGGER.warn("ipcHandle {} restored", delegate); listener.ipcHandleRestored(delegate); - return delegate; } -- To view, visit https://asterix-gerrit.ics.uci.edu/2287 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: merged Gerrit-Change-Id: I916939f3419cdc6983253b4f6d8b46756fa13e69 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow <[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]>
