From aace111a151b8065965ae10987c0cac01c517742 Mon Sep 17 00:00:00 2001
From: Carter Kozak <ckozak@palantir.com>
Date: Mon, 2 Oct 2023 11:17:31 -0400
Subject: [PATCH] suppress

---
 .../InstrumentedPoolingHttpClientConnectionManager.java    | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/dialogue-apache-hc5-client/src/main/java/com/palantir/dialogue/hc5/InstrumentedPoolingHttpClientConnectionManager.java b/dialogue-apache-hc5-client/src/main/java/com/palantir/dialogue/hc5/InstrumentedPoolingHttpClientConnectionManager.java
index 7659f8f9c..fca63509d 100644
--- a/dialogue-apache-hc5-client/src/main/java/com/palantir/dialogue/hc5/InstrumentedPoolingHttpClientConnectionManager.java
+++ b/dialogue-apache-hc5-client/src/main/java/com/palantir/dialogue/hc5/InstrumentedPoolingHttpClientConnectionManager.java
@@ -24,8 +24,8 @@
 import com.palantir.tracing.CloseableTracer;
 import com.palantir.tritium.metrics.registry.TaggedMetricRegistry;
 import java.io.IOException;
-import java.time.Duration;
 import java.util.Set;
+import java.util.concurrent.TimeUnit;
 import org.apache.hc.client5.http.HttpRoute;
 import org.apache.hc.client5.http.impl.io.PoolingHttpClientConnectionManager;
 import org.apache.hc.client5.http.io.ConnectionEndpoint;
@@ -39,6 +39,7 @@
 import org.apache.hc.core5.util.Timeout;
 
 /** A simple wrapper around a {@link PoolingHttpClientConnectionManager} which provides instrumentation. */
+@SuppressWarnings("PreferJavaTimeOverload") // perf sensitive
 final class InstrumentedPoolingHttpClientConnectionManager
         implements HttpClientConnectionManager, ConnPoolControl<HttpRoute> {
 
@@ -121,9 +122,9 @@ public void connect(ConnectionEndpoint endpoint, TimeValue connectTimeout, HttpC
         long beginNanos = System.nanoTime();
         try (CloseableTracer ignored = CloseableTracer.startSpan("Dialogue ConnectionManager.connect")) {
             manager.connect(endpoint, connectTimeout, context);
-            connectTimerSuccess.update(Duration.ofNanos(System.nanoTime() - beginNanos));
+            connectTimerSuccess.update(System.nanoTime() - beginNanos, TimeUnit.NANOSECONDS);
         } catch (Throwable throwable) {
-            connectTimerFailure.update(Duration.ofNanos(System.nanoTime() - beginNanos));
+            connectTimerFailure.update(System.nanoTime() - beginNanos, TimeUnit.NANOSECONDS);
             DialogueClientMetrics.of(registry)
                     .connectionCreateError()
                     .clientName(clientName)