summaryrefslogtreecommitdiffstats
path: root/metrics-proxy/src/main/java/ai/vespa/metricsproxy/http/GenericMetricsHandler.java
diff options
context:
space:
mode:
Diffstat (limited to 'metrics-proxy/src/main/java/ai/vespa/metricsproxy/http/GenericMetricsHandler.java')
-rw-r--r--metrics-proxy/src/main/java/ai/vespa/metricsproxy/http/GenericMetricsHandler.java35
1 files changed, 3 insertions, 32 deletions
diff --git a/metrics-proxy/src/main/java/ai/vespa/metricsproxy/http/GenericMetricsHandler.java b/metrics-proxy/src/main/java/ai/vespa/metricsproxy/http/GenericMetricsHandler.java
index f61a96917a9..cadfc053b94 100644
--- a/metrics-proxy/src/main/java/ai/vespa/metricsproxy/http/GenericMetricsHandler.java
+++ b/metrics-proxy/src/main/java/ai/vespa/metricsproxy/http/GenericMetricsHandler.java
@@ -4,9 +4,7 @@
package ai.vespa.metricsproxy.http;
-import ai.vespa.metricsproxy.core.MetricsConsumers;
import ai.vespa.metricsproxy.core.MetricsManager;
-import ai.vespa.metricsproxy.metric.model.ConsumerId;
import ai.vespa.metricsproxy.metric.model.MetricsPacket;
import ai.vespa.metricsproxy.metric.model.json.JsonRenderingException;
import ai.vespa.metricsproxy.service.VespaServices;
@@ -21,33 +19,22 @@ import java.nio.charset.Charset;
import java.time.Instant;
import java.util.List;
import java.util.concurrent.Executor;
-import java.util.logging.Logger;
-import java.util.stream.Collectors;
-import static ai.vespa.metricsproxy.metric.model.ConsumerId.toConsumerId;
import static ai.vespa.metricsproxy.metric.model.json.GenericJsonUtil.toGenericJsonModel;
/**
- * Http handler that exposes the generic metrics format.
+ * Handler exposing the generic metrics format via http.
*
* @author gjoranv
*/
public class GenericMetricsHandler extends ThreadedHttpRequestHandler {
- private static final Logger log = Logger.getLogger(GenericMetricsHandler.class.getName());
- public static final ConsumerId DEFAULT_PUBLIC_CONSUMER_ID = toConsumerId("default");
-
- private final MetricsConsumers metricsConsumers;
private final MetricsManager metricsManager;
private final VespaServices vespaServices;
@Inject
- public GenericMetricsHandler(Executor executor,
- MetricsManager metricsManager,
- VespaServices vespaServices,
- MetricsConsumers metricsConsumers) {
+ public GenericMetricsHandler(Executor executor, MetricsManager metricsManager, VespaServices vespaServices) {
super(executor);
- this.metricsConsumers = metricsConsumers;
this.metricsManager = metricsManager;
this.vespaServices = vespaServices;
}
@@ -55,29 +42,13 @@ public class GenericMetricsHandler extends ThreadedHttpRequestHandler {
@Override
public HttpResponse handle(HttpRequest request) {
try {
- ConsumerId consumer = getConsumerOrDefault(request.getProperty("consumer"));
-
- List<MetricsPacket> metrics = metricsManager.getMetrics(vespaServices.getVespaServices(), Instant.now())
- .stream()
- .filter(metricsPacket -> metricsPacket.consumers().contains(consumer))
- .collect(Collectors.toList());
+ List<MetricsPacket> metrics = metricsManager.getMetrics(vespaServices.getVespaServices(), Instant.now());
return new Response(200, toGenericJsonModel(metrics).serialize());
} catch (JsonRenderingException e) {
return new Response(500, e.getMessageAsJson());
}
}
- private ConsumerId getConsumerOrDefault(String consumer) {
- if (consumer == null) return DEFAULT_PUBLIC_CONSUMER_ID;
-
- ConsumerId consumerId = toConsumerId(consumer);
- if (! metricsConsumers.getAllConsumers().contains(consumerId)) {
- log.info("No consumer with id '" + consumer + "' - using the default consumer instead.");
- return DEFAULT_PUBLIC_CONSUMER_ID;
- }
- return consumerId;
- }
-
private static class Response extends HttpResponse {
private final byte[] data;