Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11639 )
Change subject: [compaction] KUDU-2056: Expose a metric for how much a tablet needs to be compacted ...................................................................... [compaction] KUDU-2056: Expose a metric for how much a tablet needs to be compacted This adds a new metric 'average_diskrowset_height' that reflects how uncompacted a tablet replica is. This metric is obtained by integrating the height function with respect to the by-data-size probability distribution used by the compaction policy. To implement the integration, I piggy-backed on the function that computes the CDF for the rowset layout, since computing the integral requires computing the CDF and it seemed wasteful to first compute the CDF, then go through an almost entirely similar bit of logic to compute the average height. Change-Id: I98493b901d37bb278167ba2fe98d322a86a1f0f9 Reviewed-on: http://gerrit.cloudera.org:8080/11639 Reviewed-by: Andrew Wong <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/tablet/compaction_policy-test.cc M src/kudu/tablet/compaction_policy.cc M src/kudu/tablet/rowset_info.cc M src/kudu/tablet/rowset_info.h M src/kudu/tablet/tablet.cc M src/kudu/tablet/tablet_metrics.cc M src/kudu/tablet/tablet_metrics.h 7 files changed, 287 insertions(+), 53 deletions(-) Approvals: Andrew Wong: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/11639 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I98493b901d37bb278167ba2fe98d322a86a1f0f9 Gerrit-Change-Number: 11639 Gerrit-PatchSet: 4 Gerrit-Owner: Will Berkeley <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Reviewer: Will Berkeley <[email protected]>
