Daniel Becker has uploaded this change for review. ( http://gerrit.cloudera.org:8080/21026
Change subject: WIP - IMPALA-12609: Implement SHOW TABLES IN statement to list Iceberg Metadata tables ...................................................................... WIP - IMPALA-12609: Implement SHOW TABLES IN statement to list Iceberg Metadata tables After this change, the SHOW TABLES IN statement can be used to list all the available metadata tables of an Iceberg table. Note that similarly to querying the contents of Iceberg metadata tables, this also requires fully qualified paths, e.g. SHOW TABLES IN functional_parquet.iceberg_query_metadata; works, but USE functional_parquet; SHOW TABLES IN iceberg_query_metadata; does not work. The available metadata tables for all Iceberg tables are the same, corresponding to the values of the org.apache.iceberg.MetadataTableType enum, so there is actually no need to pass the name of the regular table for which the metadata table list is requested through Thrift. This change, however, does send the table name because this way - we can check also at the point of generating the list of metadata tables that the table is an Iceberg table - if we add support for metadata tables for other table formats, the table name/path will be necessary to determine the correct list of metadata tables Testing: - added and updated tests in ParserTest, AnalyzeDDLTest, ToSqlTest and AuthorizationStmtTest. TODO: privileges should probably be tested more thoroughly. Change-Id: Ide10ccf10fc0abf5c270119ba7092c67e712ec49 --- M be/src/service/client-request-state.cc M be/src/service/frontend.cc M be/src/service/frontend.h M be/src/service/impala-http-handler.cc M common/thrift/Frontend.thrift M fe/src/main/cup/sql-parser.cup M fe/src/main/java/org/apache/impala/analysis/ShowTablesStmt.java M fe/src/main/java/org/apache/impala/service/Frontend.java M fe/src/main/java/org/apache/impala/service/JniFrontend.java M fe/src/main/java/org/apache/impala/service/MetadataOp.java M fe/src/test/java/org/apache/impala/analysis/AnalyzeDDLTest.java M fe/src/test/java/org/apache/impala/analysis/ParserTest.java M fe/src/test/java/org/apache/impala/analysis/ToSqlTest.java M fe/src/test/java/org/apache/impala/authorization/AuthorizationStmtTest.java 14 files changed, 227 insertions(+), 62 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/26/21026/1 -- To view, visit http://gerrit.cloudera.org:8080/21026 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ide10ccf10fc0abf5c270119ba7092c67e712ec49 Gerrit-Change-Number: 21026 Gerrit-PatchSet: 1 Gerrit-Owner: Daniel Becker <[email protected]> Gerrit-Reviewer: Noemi Pap-Takacs <[email protected]> Gerrit-Reviewer: Tamas Mate <[email protected]> Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
