From 52b6d5c438b795b76b9fcebfb1f1ab59b9080414 Mon Sep 17 00:00:00 2001 From: Patrick Koenig Date: Mon, 30 Sep 2024 16:00:07 -0400 Subject: [PATCH] Allow non-constant endpoints to opt-in to timings --- .../main/java/com/palantir/dialogue/core/DialogueChannel.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/dialogue-core/src/main/java/com/palantir/dialogue/core/DialogueChannel.java b/dialogue-core/src/main/java/com/palantir/dialogue/core/DialogueChannel.java index e4424d0e8..efd5a0ac5 100644 --- a/dialogue-core/src/main/java/com/palantir/dialogue/core/DialogueChannel.java +++ b/dialogue-core/src/main/java/com/palantir/dialogue/core/DialogueChannel.java @@ -253,7 +253,8 @@ private static EndpointChannelFactory createEndpointChannelFactory(Channel multi channel = new RangeAcceptsIdentityEncodingChannel(channel); channel = ContentEncodingChannel.of(channel, endpoint); channel = TracedChannel.create(cf, channel, endpoint); - if (ChannelToEndpointChannel.isConstant(endpoint)) { + if (ChannelToEndpointChannel.isConstant(endpoint) + || endpoint.tags().contains("dialogue-enable-endpoint-timing")) { // Avoid producing metrics for non-constant endpoints which may produce // high cardinality. channel = TimingEndpointChannel.create(cf, channel, endpoint);