[email protected] has posted comments on this change. ( http://gerrit.cloudera.org:8080/17727 )
Change subject: IMPALA-10797: Frontend changes to enable 'stored as JSONFILE' ...................................................................... Patch Set 6: (3 comments) > (3 comments) http://gerrit.cloudera.org:8080/#/c/17727/5/fe/src/main/java/org/apache/impala/analysis/AlterTableSetRowFormatStmt.java File fe/src/main/java/org/apache/impala/analysis/AlterTableSetRowFormatStmt.java: http://gerrit.cloudera.org:8080/#/c/17727/5/fe/src/main/java/org/apache/impala/analysis/AlterTableSetRowFormatStmt.java@85 PS5, Line 85: sd.getInputFormat(), sd.getSerdeInfo().getSerializationLib()); > We can extract "((FeFsTable) tbl).getMetaStoreTable().getSd()" into a varia Done http://gerrit.cloudera.org:8080/#/c/17727/5/testdata/workloads/functional-query/queries/QueryTest/show-create-table.test File testdata/workloads/functional-query/queries/QueryTest/show-create-table.test: http://gerrit.cloudera.org:8080/#/c/17727/5/testdata/workloads/functional-query/queries/QueryTest/show-create-table.test@24 PS5, Line 24: json_ > Is this correct to reuse "test1"? We already create a "test1" table above. Done http://gerrit.cloudera.org:8080/#/c/17727/5/testdata/workloads/functional-query/queries/QueryTest/show-create-table.test@25 PS5, Line 25: id INT > Could you add all supported types? Added ORC,AVRO,KUDU. PARQUET, SEQUENCEFILE, TEXT, ICEBERG already have tests in the file. -- To view, visit http://gerrit.cloudera.org:8080/17727 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I5b8cb2f59df3af09902b49d3bdac16c19954b305 Gerrit-Change-Number: 17727 Gerrit-PatchSet: 6 Gerrit-Owner: Anonymous Coward <[email protected]> Gerrit-Reviewer: Anonymous Coward <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Comment-Date: Tue, 10 Aug 2021 16:42:13 +0000 Gerrit-HasComments: Yes
