Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12282 )
Change subject: IMPALA-6050: Query profiles should indicate storage layer(s) used ...................................................................... Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/12282/1/fe/src/main/java/org/apache/impala/catalog/FeFsTable.java File fe/src/main/java/org/apache/impala/catalog/FeFsTable.java: http://gerrit.cloudera.org:8080/#/c/12282/1/fe/src/main/java/org/apache/impala/catalog/FeFsTable.java@344 PS1, Line 344: Map<HdfsScanNode.SampledPartitionMetadata, List<FileDescriptor>> result = new HashMap<>(); line too long (96 > 90) http://gerrit.cloudera.org:8080/#/c/12282/1/fe/src/main/java/org/apache/impala/planner/HdfsScanNode.java File fe/src/main/java/org/apache/impala/planner/HdfsScanNode.java: http://gerrit.cloudera.org:8080/#/c/12282/1/fe/src/main/java/org/apache/impala/planner/HdfsScanNode.java@799 PS1, Line 799: * Initializes members with information about files and scan ranges, e.g. totalFilesPerFs_, line too long (93 > 90) http://gerrit.cloudera.org:8080/#/c/12282/1/fe/src/main/java/org/apache/impala/planner/HdfsScanNode.java@1072 PS1, Line 1072: // The extrapolated row count is based on the 'totalBytesPerFs_' which already accounts line too long (93 > 90) -- To view, visit http://gerrit.cloudera.org:8080/12282 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I4b1b4a1bc1a24e9614e3b4dc5a61dc96d075d1c3 Gerrit-Change-Number: 12282 Gerrit-PatchSet: 1 Gerrit-Owner: Sahil Takiar <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Comment-Date: Mon, 28 Jan 2019 16:41:13 +0000 Gerrit-HasComments: Yes
