Hello Tidy Bot, Kudu Jenkins, Adar Dembo, Hao Hao,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/11493
to look at the new patch set (#2).
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, 232 insertions(+), 104 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/93/11493/2
--
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: newpatchset
Gerrit-Change-Id: I6f6f843f42b37cb1170df03da01fc0790fe94acb
Gerrit-Change-Number: 11493
Gerrit-PatchSet: 2
Gerrit-Owner: Dan Burkert <danburk...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Hao Hao <hao....@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot
