From 6189207f8b3fa51e29b903e3322e654d5a06ebdc Mon Sep 17 00:00:00 2001
From: Arne Juul <arnej@yahoo-inc.com>
Date: Tue, 25 Sep 2018 10:28:21 +0000
Subject: fix lost handler metrics

* since all these classes use a shared FeedContext instance we must make
  sure that nobody ever uses NullFeedMetric (except unit tests & VespaFeeder).
---
 .../src/main/java/com/yahoo/feedhandler/NullFeedMetric.java        | 7 +++++++
 1 file changed, 7 insertions(+)

(limited to 'vespaclient-core/src/main')

diff --git a/vespaclient-core/src/main/java/com/yahoo/feedhandler/NullFeedMetric.java b/vespaclient-core/src/main/java/com/yahoo/feedhandler/NullFeedMetric.java
index b8363ace62d..2940c0fcc44 100644
--- a/vespaclient-core/src/main/java/com/yahoo/feedhandler/NullFeedMetric.java
+++ b/vespaclient-core/src/main/java/com/yahoo/feedhandler/NullFeedMetric.java
@@ -9,6 +9,13 @@ import java.util.Map;
  * @since 5.1.20
  */
 public final class NullFeedMetric implements Metric {
+
+    public NullFeedMetric(boolean flag) {
+        if (!flag) {
+            throw new IllegalArgumentException("must set flag allowing to throw away metrics");
+        }
+    }
+
     @Override
     public void set(String key, Number val, Context ctx) {
     }
-- 
cgit v1.2.3