Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/19201 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/19201 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- 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(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- 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: merged Gerrit-Change-Id: If0e84e7756ce92af48fa55672157caafcd396547 Gerrit-Change-Number: 19201 Gerrit-PatchSet: 6 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]>
