Sailesh Mukil has uploaded a new change for review. http://gerrit.cloudera.org:8080/4724
Change subject: KUDU-1700: Debug build will not fail gracefully on Messenger::Init() failure ...................................................................... KUDU-1700: Debug build will not fail gracefully on Messenger::Init() failure If Messenger::Init() fails in a debug build, a CHECK() will happen on the destruction of the Messenger object. This patch reorders some code in the MessengerBuilder::Build() functions to make sure that doesn't happen. This case was probably never encountered as Messenger::Init() currently has a very low chance of failing. However, in the future, as more things may get added on to the function, this issue might show up more often. A more detailed explanation is given in the JIRA. Change-Id: Id6021587c746af53305b3f601bb1bcc19f63eab0 --- M src/kudu/rpc/messenger.cc 1 file changed, 3 insertions(+), 3 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/24/4724/1 -- To view, visit http://gerrit.cloudera.org:8080/4724 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Id6021587c746af53305b3f601bb1bcc19f63eab0 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Sailesh Mukil <[email protected]>
