Yingchun Lai has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/14903 )

Change subject: [metrics] Support SUM, MAX and MIN merge types for AtomicGauge
......................................................................


Patch Set 2:

> Patch Set 2:
>
> (4 comments)
>
> Will you be using this in follow-up commits? Which metrics are you thinking 
> of changing?

Yes. I want to add a metric like 'last_consult_time' to tablets, represent the 
last write or scan timestamp of a tablet. We can judge whether a tablet is 
'hot' or 'cold' by this metric, and it need a 'MAX' type when merge into table 
level metric.


--
To view, visit http://gerrit.cloudera.org:8080/14903
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I1acad40c2b73cfeb16e0dc4ad5ffba4f19167c81
Gerrit-Change-Number: 14903
Gerrit-PatchSet: 2
Gerrit-Owner: Yingchun Lai <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Yingchun Lai <[email protected]>
Gerrit-Comment-Date: Sat, 14 Dec 2019 00:43:23 +0000
Gerrit-HasComments: No

Reply via email to