diff options
| -rw-r--r-- | java/com/google/gerrit/metrics/MetricMaker.java | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/java/com/google/gerrit/metrics/MetricMaker.java b/java/com/google/gerrit/metrics/MetricMaker.java index 42ec8a09a9..618d421e2e 100644 --- a/java/com/google/gerrit/metrics/MetricMaker.java +++ b/java/com/google/gerrit/metrics/MetricMaker.java @@ -78,14 +78,15 @@ public abstract class MetricMaker { * @param name unique name of the metric. * @param value only value of the metric. * @param desc description of the metric. + * @return registration handle */ - public <V> void newConstantMetric(String name, V value, Description desc) { + public <V> RegistrationHandle newConstantMetric(String name, V value, Description desc) { desc.setConstant(); @SuppressWarnings("unchecked") Class<V> type = (Class<V>) value.getClass(); CallbackMetric0<V> metric = newCallbackMetric(name, type, desc); - newTrigger(metric, () -> metric.set(value)); + return newTrigger(metric, () -> metric.set(value)); } /** @@ -107,11 +108,12 @@ public abstract class MetricMaker { * @param valueClass type of value recorded by the metric. * @param desc description of the metric. * @param trigger function to compute the value of the metric. + * @return registration handle */ - public <V> void newCallbackMetric( + public <V> RegistrationHandle newCallbackMetric( String name, Class<V> valueClass, Description desc, Supplier<V> trigger) { CallbackMetric0<V> metric = newCallbackMetric(name, valueClass, desc); - newTrigger(metric, () -> metric.set(trigger.get())); + return newTrigger(metric, () -> metric.set(trigger.get())); } /** |