Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17864 )
Change subject: IMPALA-10928: Upgrade source of Kudu's EasyCurl in kudu/util ...................................................................... IMPALA-10928: Upgrade source of Kudu's EasyCurl in kudu/util We are going to use Kudu's EasyCurl wrapper to download file in Impala. There are some enhancements for Kudu's EasyCurl in Kudu upstream since the last sync of kudu/util with Kudu repo. This patch upgraded source code for Kudu's EasyCurl wrapper (curl_util.cc and curl_util.h in kudu/util) from the Kudu upstream (commit hash: 37bc427a60, b417479266, a33b0951d9, 9050941663, e371d80f3e). Note that curl_util.cc/curl_util.h have dependency on libcurl, which is not integrated in Impala. curl_util.cc is not built with current kudu/util/CMakeLists.txt. We will integrate curl_util.cc/curl_util.h when we integrate libcurl in following patch. Testing: - There is no impact for Impala tests. Change-Id: I51fea9ff4de767bfda2a4ebcd3744d63440adf14 Reviewed-on: http://gerrit.cloudera.org:8080/17864 Reviewed-by: Joe McDonnell <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M be/src/kudu/util/curl_util-test.cc M be/src/kudu/util/curl_util.cc M be/src/kudu/util/curl_util.h 3 files changed, 155 insertions(+), 52 deletions(-) Approvals: Joe McDonnell: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/17864 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I51fea9ff4de767bfda2a4ebcd3744d63440adf14 Gerrit-Change-Number: 17864 Gerrit-PatchSet: 3 Gerrit-Owner: Wenzhe Zhou <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]>
