Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12898 )
Change subject: [curl] Fix error status of timeout ...................................................................... [curl] Fix error status of timeout This commit fix the returned status of class EasyCurl functions, and also add unit test. Some other unit test utils like class PeriodicWebUIChecker depends on the 'Timeout' status of curl.FetchURL(). Change-Id: Ib8fab1bd1a34b06624db4fd51d782a213f93321c Reviewed-on: http://gerrit.cloudera.org:8080/12898 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <[email protected]> --- M src/kudu/util/CMakeLists.txt A src/kudu/util/curl_util-test.cc M src/kudu/util/curl_util.cc 3 files changed, 51 insertions(+), 0 deletions(-) Approvals: Kudu Jenkins: Verified Adar Dembo: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/12898 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ib8fab1bd1a34b06624db4fd51d782a213f93321c Gerrit-Change-Number: 12898 Gerrit-PatchSet: 6 Gerrit-Owner: Yingchun Lai <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yingchun Lai <[email protected]>
