Riza Suminto has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/22297 )
Change subject: IMPALA-13641: Lazily init Parquet column read counters ...................................................................... IMPALA-13641: Lazily init Parquet column read counters ParquetUncompressedBytesReadPerColumn and ParquetCompressedBytesReadPerColumn exist in runtime profile even when no parquet file is read (all scan text files). This patch lazily init those counters only if HdfsScanNodeBase::bytes_read_per_col_ is not empty. Testing: - Run and pass TestParquet::test_bytes_read_per_column. - Run TestTpcdsInsert and confirm no Parquet specific counters exist when reading TEXTFILE table. Change-Id: I8ba767b69b8c432f0eb954aa54f86876b329160c Reviewed-on: http://gerrit.cloudera.org:8080/22297 Reviewed-by: Michael Smith <michael.sm...@cloudera.com> Reviewed-by: Csaba Ringhofer <csringho...@cloudera.com> Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> --- M be/src/exec/hdfs-scan-node-base.cc M be/src/exec/hdfs-scan-node-base.h 2 files changed, 15 insertions(+), 18 deletions(-) Approvals: Michael Smith: Looks good to me, but someone else must approve Csaba Ringhofer: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/22297 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I8ba767b69b8c432f0eb954aa54f86876b329160c Gerrit-Change-Number: 22297 Gerrit-PatchSet: 3 Gerrit-Owner: Riza Suminto <riza.sumi...@cloudera.com> Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Michael Smith <michael.sm...@cloudera.com> Gerrit-Reviewer: Riza Suminto <riza.sumi...@cloudera.com>
