Todd Lipcon has submitted this change and it was merged.

Change subject: client/sample.cc: fixed a couple of crashes
......................................................................


client/sample.cc: fixed a couple of crashes

Fixed 'pure virtual method called' error with subsequent SIGABRT
termination due to attempt to access destroyed log callback object
while logging some messages from terminating reactor threads.

Fixed issue with an attempt to access non-existing element
of pending errors array with subsequent SIGSEGV termination.

Change-Id: Ic5fa3b812e6402a113bf5e432a3a451dc4cc3735
Reviewed-on: http://gerrit.cloudera.org:8080/3685
Tested-by: Kudu Jenkins
Reviewed-by: Todd Lipcon <[email protected]>
---
M src/kudu/client/samples/sample.cc
1 file changed, 43 insertions(+), 24 deletions(-)

Approvals:
  Todd Lipcon: Looks good to me, approved
  Kudu Jenkins: Verified



-- 
To view, visit http://gerrit.cloudera.org:8080/3685
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic5fa3b812e6402a113bf5e432a3a451dc4cc3735
Gerrit-PatchSet: 4
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>

Reply via email to