Dan Burkert has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11493 )
Change subject: Add thrift module for common thrift utilities ...................................................................... Add thrift module for common thrift utilities The HMS patch series inlined all necessary Thrift utilities into the hms module, since it was the only use of Thrift in the codebase. Now that we're also planning on having a Sentry client it makes sense to properly abstract the common Thrift code into its own shared module. Change-Id: I6f6f843f42b37cb1170df03da01fc0790fe94acb Reviewed-on: http://gerrit.cloudera.org:8080/11493 Reviewed-by: Adar Dembo <a...@cloudera.com> Tested-by: Kudu Jenkins --- M CMakeLists.txt M src/kudu/hms/CMakeLists.txt M src/kudu/hms/hms_catalog-test.cc M src/kudu/hms/hms_catalog.cc M src/kudu/hms/hms_catalog.h M src/kudu/hms/hms_client-test.cc M src/kudu/hms/hms_client.cc M src/kudu/hms/hms_client.h M src/kudu/integration-tests/master_hms-itest.cc M src/kudu/mini-cluster/external_mini_cluster-test.cc A src/kudu/thrift/CMakeLists.txt A src/kudu/thrift/client.cc A src/kudu/thrift/client.h R src/kudu/thrift/sasl_client_transport.cc R src/kudu/thrift/sasl_client_transport.h M src/kudu/tools/kudu-tool-test.cc 16 files changed, 233 insertions(+), 105 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/11493 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I6f6f843f42b37cb1170df03da01fc0790fe94acb Gerrit-Change-Number: 11493 Gerrit-PatchSet: 4 Gerrit-Owner: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: Hao Hao <hao....@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot
