Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15100 )
Change subject: [tests] minor clean up on linked_list test ...................................................................... [tests] minor clean up on linked_list test This patch updates the linked_list test to output more information in case if the curl fetcher thread fails to retrieve data from one of the monitored URLs. I also updated the code to handle non-OK status returned from KuduScanner::GetCurrentServer(), replaced gscoped_ptr with std::unique_ptr, and did other minor updates. The motivation for this change is get more details on the error if it happens. Change-Id: Ic92c3a41cb3f37426c32a2c61a11c400cf855cf7 Reviewed-on: http://gerrit.cloudera.org:8080/15100 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <[email protected]> --- M src/kudu/integration-tests/linked_list-test-util.h 1 file changed, 25 insertions(+), 21 deletions(-) Approvals: Kudu Jenkins: Verified Adar Dembo: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/15100 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ic92c3a41cb3f37426c32a2c61a11c400cf855cf7 Gerrit-Change-Number: 15100 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241)
