Vihang Karajgaonkar has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16121 )

Change subject: IMPALA-9907: Fix NullPointerException in 
ParallelFileMetadataLoader's load() method
......................................................................


Patch Set 1: Code-Review+2

(1 comment)

Thanks for fixing this.

http://gerrit.cloudera.org:8080/#/c/16121/1/fe/src/main/java/org/apache/impala/catalog/ParallelFileMetadataLoader.java
File fe/src/main/java/org/apache/impala/catalog/ParallelFileMetadataLoader.java:

http://gerrit.cloudera.org:8080/#/c/16121/1/fe/src/main/java/org/apache/impala/catalog/ParallelFileMetadataLoader.java@166
PS1, Line 166: loader,
             :             pool.submit(() -> { loader.load(); return null; })));
nit, can be moved to earlier line.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I4322a18a1f07e4106789591b80d25105c565a2b6
Gerrit-Change-Number: 16121
Gerrit-PatchSet: 1
Gerrit-Owner: Zoltan Borok-Nagy <borokna...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Vihang Karajgaonkar <vih...@cloudera.com>
Gerrit-Comment-Date: Mon, 29 Jun 2020 17:36:57 +0000
Gerrit-HasComments: Yes

Reply via email to