Yingchun Lai has posted comments on this change. ( http://gerrit.cloudera.org:8080/14128 )
Change subject: [metrics] Add a metric to count merged entities when merge metrics ...................................................................... Patch Set 4: (3 comments) http://gerrit.cloudera.org:8080/#/c/14128/3//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/14128/3//COMMIT_MSG@13 PS3, Line 13: check whether a table is balanced or not by thirdparty monitor system. > Thanks, that's useful to know. Maybe you can find a way to incorporate this Done http://gerrit.cloudera.org:8080/#/c/14128/4/src/kudu/util/metrics.h File src/kudu/util/metrics.h: http://gerrit.cloudera.org:8080/#/c/14128/4/src/kudu/util/metrics.h@737 PS4, Line 737: // Invalidate 'm_epoch_', then this metric is invisible until its value changed. > Nit: "Invalidate 'm_epoch_', causing this metric to be invisible until its Done http://gerrit.cloudera.org:8080/#/c/14128/2/src/kudu/util/metrics.cc File src/kudu/util/metrics.cc: http://gerrit.cloudera.org:8080/#/c/14128/2/src/kudu/util/metrics.cc@479 PS2, Line 479: // metrics, we should keep them around until the next poll. > Hmm, I don't see this change done: I still see just the one static metric p Done -- To view, visit http://gerrit.cloudera.org:8080/14128 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I652b457a2df73414f95f5d1d5efaa003cc262bd1 Gerrit-Change-Number: 14128 Gerrit-PatchSet: 4 Gerrit-Owner: Yingchun Lai <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yingchun Lai <[email protected]> Gerrit-Comment-Date: Wed, 04 Sep 2019 05:33:19 +0000 Gerrit-HasComments: Yes
