Hello Adar Dembo, Hao Hao,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/11493
to review the following change.
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
---
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(+), 100 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/93/11493/1
--
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: newchange
Gerrit-Change-Id: I6f6f843f42b37cb1170df03da01fc0790fe94acb
Gerrit-Change-Number: 11493
Gerrit-PatchSet: 1
Gerrit-Owner: Dan Burkert <danburk...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Hao Hao <hao....@cloudera.com>
