Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16143 )

Change subject: IMPALA-9741: Support quering Icebreg table by impala
......................................................................


Patch Set 9:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/16143/9/fe/src/main/java/org/apache/impala/catalog/IcebergTable.java
File fe/src/main/java/org/apache/impala/catalog/IcebergTable.java:

http://gerrit.cloudera.org:8080/#/c/16143/9/fe/src/main/java/org/apache/impala/catalog/IcebergTable.java@208
PS9, Line 208:         pathMD5ToFileDescMap_ = 
Utils.loadAllPartition(msTable_.getSd().getLocation(), this);
line too long (93 > 90)


http://gerrit.cloudera.org:8080/#/c/16143/9/fe/src/main/java/org/apache/impala/catalog/IcebergTable.java@277
PS9, Line 277:     pathMD5ToFileDescMap_ = 
loadFileDescFromThrift(ticeberg.getPath_md5_to_file_descriptors());
line too long (95 > 90)


http://gerrit.cloudera.org:8080/#/c/16143/9/fe/src/main/java/org/apache/impala/planner/IcebergScanNode.java
File fe/src/main/java/org/apache/impala/planner/IcebergScanNode.java:

http://gerrit.cloudera.org:8080/#/c/16143/9/fe/src/main/java/org/apache/impala/planner/IcebergScanNode.java@102
PS9, Line 102:           
icebergTable_.getPathMD5ToFileDescMap().get(IcebergUtil.getDataFileMD5(dataFile));
line too long (92 > 90)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I856cfee4f3397d1a89cf17650e8d4fbfe1f2b006
Gerrit-Change-Number: 16143
Gerrit-PatchSet: 9
Gerrit-Owner: wangsheng <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
Gerrit-Reviewer: wangsheng <[email protected]>
Gerrit-Comment-Date: Thu, 16 Jul 2020 06:29:16 +0000
Gerrit-HasComments: Yes

Reply via email to