KeDeng has posted comments on this change. ( http://gerrit.cloudera.org:8080/21291 )
Change subject: [metrics] Add tablet level metrics for scans op time ...................................................................... Patch Set 5: (3 comments) Thanks for your reviews. http://gerrit.cloudera.org:8080/#/c/21291/4/src/kudu/tablet/tablet_metrics.h File src/kudu/tablet/tablet_metrics.h: http://gerrit.cloudera.org:8080/#/c/21291/4/src/kudu/tablet/tablet_metrics.h@68 PS4, Line 68: scoped_refptr<Histogram> tablet_scans_duration_wall_time; : scoped_refptr<Histogram> tablet_scans_duration_system_time; > That's interesting: the system and the wall times are collected, but not th I have added statistics on the user time. Thanks four your advice. http://gerrit.cloudera.org:8080/#/c/21291/4/src/kudu/tablet/tablet_metrics.cc File src/kudu/tablet/tablet_metrics.cc: http://gerrit.cloudera.org:8080/#/c/21291/4/src/kudu/tablet/tablet_metrics.cc@160 PS4, Line 160: kDebu > Here and below: I'm not sure this metric is anything but extra information Done http://gerrit.cloudera.org:8080/#/c/21291/4/src/kudu/tserver/tablet_service.cc File src/kudu/tserver/tablet_service.cc: http://gerrit.cloudera.org:8080/#/c/21291/4/src/kudu/tserver/tablet_service.cc@3215 PS4, Line 3215: unreg_scanner.Cancel(); > There are several earlier returns from this method where scanner times are I have updated the timing of the metrics update. Thank you for your suggestion. -- To view, visit http://gerrit.cloudera.org:8080/21291 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I8f490cfb6f37aef60b34697100fb502374fcc503 Gerrit-Change-Number: 21291 Gerrit-PatchSet: 5 Gerrit-Owner: KeDeng <kdeng...@gmail.com> Gerrit-Reviewer: Abhishek Chennaka <achenn...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <ale...@apache.org> Gerrit-Reviewer: KeDeng <kdeng...@gmail.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yingchun Lai <laiyingc...@apache.org> Gerrit-Comment-Date: Wed, 15 May 2024 08:30:10 +0000 Gerrit-HasComments: Yes
