Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8147 )
Change subject: IMPALA-5448: fix invalid number of splits reported in Parquet scan node ...................................................................... Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/8147/5/testdata/workloads/functional-query/queries/QueryTest/hdfs_parquet_scan_node_profile.test File testdata/workloads/functional-query/queries/QueryTest/hdfs_parquet_scan_node_profile.test: http://gerrit.cloudera.org:8080/#/c/8147/5/testdata/workloads/functional-query/queries/QueryTest/hdfs_parquet_scan_node_profile.test@12 PS5, Line 12: row_regex: .*File Formats: (PARQUET/GZIP:1 PARQUET/SNAPPY:1|PARQUET/SNAPPY:1 PARQUET/GZIP:1) Is the | needed? The counts are stored in a sorted std::map so I expected the order to be deterministic. -- To view, visit http://gerrit.cloudera.org:8080/8147 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iaacc2d775032f5707061e704f12e0a63cde695d1 Gerrit-Change-Number: 8147 Gerrit-PatchSet: 5 Gerrit-Owner: Quanlong Huang <[email protected]> Gerrit-Reviewer: Mostafa Mokhtar <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-Comment-Date: Fri, 06 Oct 2017 00:15:43 +0000 Gerrit-HasComments: Yes
