Dan Burkert has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11524 )
Change subject: KUDU-2541: Fill out basic sentry client API ...................................................................... KUDU-2541: Fill out basic sentry client API This commit contains more plumbing for the upcoming Sentry integration (KUDU-428). The Sentry client class is filled out with enough methods to ensure the Thrift RPC is working correctly, as well as error conversion from Sentry and Thrift's format to Kudu's Status. Change-Id: I1004a11506ed109d1f23389ca73d95a34c32c194 Reviewed-on: http://gerrit.cloudera.org:8080/11524 Reviewed-by: Adar Dembo <a...@cloudera.com> Reviewed-by: Hao Hao <hao....@cloudera.com> Tested-by: Kudu Jenkins --- M src/kudu/sentry/CMakeLists.txt M src/kudu/sentry/mini_sentry.cc M src/kudu/sentry/mini_sentry.h M src/kudu/sentry/sentry_client-test.cc M src/kudu/sentry/sentry_client.cc M src/kudu/sentry/sentry_client.h 6 files changed, 299 insertions(+), 9 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Hao Hao: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/11524 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I1004a11506ed109d1f23389ca73d95a34c32c194 Gerrit-Change-Number: 11524 Gerrit-PatchSet: 4 Gerrit-Owner: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Andrew Wong <andrew.w...@cloudera.com> Gerrit-Reviewer: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: Hao Hao <hao....@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot