You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Quanlong Huang (Code Review)" <ge...@cloudera.org> on 2022/10/17 11:15:50 UTC

[Impala-ASF-CR] IMPALA-4052: CREATE TABLE LIKE for Kudu tables

Quanlong Huang has posted comments on this change. ( http://gerrit.cloudera.org:8080/18729 )

Change subject: IMPALA-4052: CREATE TABLE LIKE for Kudu tables
......................................................................


Patch Set 22:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/18729/18/fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java
File fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java:

http://gerrit.cloudera.org:8080/#/c/18729/18/fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java@3840
PS18, Line 3840:       Map<String, String> tableProperties = Maps
> I added the comment. When getting range partitions List<String> by kudu cli
Can we use this method? It returns a list of org.apache.kudu.client.Partition

https://github.com/apache/kudu/blob/f55c80e75848461eef21c3cdb6f43d907999bfdf/java/kudu-client/src/main/java/org/apache/kudu/client/KuduTable.java#L340



-- 
To view, visit http://gerrit.cloudera.org:8080/18729
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia3d276a6465301dbcfed17bb713aca06367d9a42
Gerrit-Change-Number: 18729
Gerrit-PatchSet: 22
Gerrit-Owner: Xiaoqing Gao <ga...@gmail.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Kurt Deschler <kd...@cloudera.com>
Gerrit-Reviewer: Quanlong Huang <hu...@gmail.com>
Gerrit-Reviewer: Wenzhe Zhou <wz...@cloudera.com>
Gerrit-Reviewer: Xiaoqing Gao <ga...@gmail.com>
Gerrit-Comment-Date: Mon, 17 Oct 2022 11:15:50 +0000
Gerrit-HasComments: Yes