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 (#3).

Change subject: IMPALA-11702: add some resource metrics for kudu scanner to 
profile
......................................................................

IMPALA-11702: add some 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].

I also update the MaterializeTupleTime counter in this patch, 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/3
--
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: 3
Gerrit-Owner: Yifan Zhang <chinazhangyi...@163.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Wenzhe Zhou <wz...@cloudera.com>

Reply via email to