Dan Burkert has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10934 )
Change subject: hms-tool: filter non-Kudu tables in the HMS ...................................................................... hms-tool: filter non-Kudu tables in the HMS Renames HmsCatalog::RetrieveTables to GetKuduTables and changes the semantics such that only Kudu tables are returned. The only caller was the HMS tool, and it only needs to inspect Kudu tables. Special APIs are added to the HMS client and HMS catalog layers that allow sending a filter to the HMS to strip out non-Kudu tables when listing tables, as well as a bulk-get API. The combination of these APIs should be significantly more efficient than issuing a get for every single table in the HMS and doing Kudu-side filtering. Also included are some style and formatting fixups in HMS tool. This patch includes no functional changes to the HMS tool. Change-Id: I5f83d2e705ea6910a9aa0a1eda0d30b5feb2607b Reviewed-on: http://gerrit.cloudera.org:8080/10934 Reviewed-by: Adar Dembo <a...@cloudera.com> Tested-by: Kudu Jenkins --- 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 M src/kudu/tools/kudu-tool-test.cc M src/kudu/tools/tool_action_hms.cc 10 files changed, 167 insertions(+), 94 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/10934 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I5f83d2e705ea6910a9aa0a1eda0d30b5feb2607b Gerrit-Change-Number: 10934 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: Todd Lipcon <t...@apache.org>
