Hello Wenzhe Zhou, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/19201
to look at the new patch set (#4).
Change subject: IMPALA-11702: add resource metrics for kudu scanner to profile
......................................................................
IMPALA-11702: add resource metrics for kudu scanner to profile
This patch adds some counters about kudu scanner resource metrics[1]
to profile. These counters are updated before we close kudu scanner
through kudu client's GetResourceMetrics interface[2].
The MaterializeTupleTime counter is also updated, it should only
include time spent materializing tuples and evaluating predicates.
Tests
- Ran some queries manually and checked the profiles.
[1]
https://github.com/apache/kudu/blob/a2fdba62d2129d0d7d0c2d0f175e8b2a6e2d2650/src/kudu/tserver/tserver.proto#L398
[2]
https://github.com/apache/kudu/blob/892bda293f238fddec47423d5c0b5be9576581f1/src/kudu/client/client.h#L2967
Change-Id: If0e84e7756ce92af48fa55672157caafcd396547
---
M be/src/exec/kudu/kudu-scan-node-base.cc
M be/src/exec/kudu/kudu-scan-node-base.h
M be/src/exec/kudu/kudu-scanner.cc
3 files changed, 72 insertions(+), 4 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/01/19201/4
--
To view, visit http://gerrit.cloudera.org:8080/19201
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: If0e84e7756ce92af48fa55672157caafcd396547
Gerrit-Change-Number: 19201
Gerrit-PatchSet: 4
Gerrit-Owner: Yifan Zhang <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Wenzhe Zhou <[email protected]>
Gerrit-Reviewer: Yifan Zhang <[email protected]>
