Hello Csaba Ringhofer, Michael Smith,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/22297
to look at the new patch set (#2).
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
---
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(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/97/22297/2
--
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: newpatchset
Gerrit-Change-Id: I8ba767b69b8c432f0eb954aa54f86876b329160c
Gerrit-Change-Number: 22297
Gerrit-PatchSet: 2
Gerrit-Owner: Riza Suminto <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>
