Excavator: Upgrades Baseline to the latest version (#1453)

build.gradle

@@ -24,7 +24,7 @@ buildscript {
         classpath 'com.palantir.gradle.externalpublish:gradle-external-publish-plugin:1.5.0'
         classpath 'com.palantir.gradle.revapi:gradle-revapi:1.5.0'
         classpath 'com.palantir.javaformat:gradle-palantir-java-format:2.3.0'
-        classpath 'com.palantir.baseline:gradle-baseline-java:4.31.0'
+        classpath 'com.palantir.baseline:gradle-baseline-java:4.32.0'
         classpath 'com.palantir.gradle.gitversion:gradle-git-version:0.12.3'
         classpath 'gradle.plugin.org.inferred:gradle-processors:3.6.0'
         classpath 'com.palantir.metricschema:gradle-metric-schema:0.8.0'

dialogue-apache-hc5-client/src/main/java/com/palantir/dialogue/hc5/InactivityValidationAwareConnectionKeepAliveStrategy.java

@@ -20,6 +20,7 @@
 import com.palantir.logsafe.SafeArg;
 import com.palantir.logsafe.logger.SafeLogger;
 import com.palantir.logsafe.logger.SafeLoggerFactory;
+import java.time.Duration;
 import java.util.Iterator;
 import java.util.Objects;
 import java.util.concurrent.atomic.AtomicReference;
@@ -49,7 +50,7 @@ final class InactivityValidationAwareConnectionKeepAliveStrategy implements Conn
     private final PoolingHttpClientConnectionManager connectionManager;
     private final String clientName;
     private final TimeValue defaultValidateAfterInactivity;
-    private final RateLimiter loggingRateLimiter = RateLimiter.create(2);
+    private final RateLimiter loggingRateLimiter = RateLimiter.create(2, Duration.ZERO);
     /**
      * This field is used for observability. It's possible, though unlikely, that the value can get out of sync
      * with the connection manager in some scenarios.

dialogue-client-verifier/src/test/java/com/palantir/verification/IntegrationTest.java

@@ -201,7 +201,7 @@ public void when_thread_is_interrupted_no_requests_are_made() {
     @Test
     public void concurrency_limiters_can_effectively_infer_server_side_ratelimits() throws Exception {
         int permitsPerSecond = 300;
-        RateLimiter rateLimiter = RateLimiter.create(permitsPerSecond);
+        RateLimiter rateLimiter = RateLimiter.create(permitsPerSecond, Duration.ZERO);
 
         undertowHandler = exchange -> {
             // These thread.sleep times are taken from a prod instance of internal-ski-product.

dialogue-core/src/main/java/com/palantir/dialogue/core/RetryOtherValidatingChannel.java

@@ -33,6 +33,7 @@
 import com.palantir.logsafe.logger.SafeLoggerFactory;
 import java.net.MalformedURLException;
 import java.net.URL;
+import java.time.Duration;
 import java.util.Optional;
 import java.util.Set;
 import java.util.function.Consumer;
@@ -41,7 +42,7 @@
 final class RetryOtherValidatingChannel implements Channel {
 
     private static final SafeLogger log = SafeLoggerFactory.get(RetryOtherValidatingChannel.class);
-    private static final RateLimiter VALIDATION_FAILED_LOGGING_LIMITER = RateLimiter.create(1);
+    private static final RateLimiter VALIDATION_FAILED_LOGGING_LIMITER = RateLimiter.create(1, Duration.ZERO);
 
     private final Channel delegate;
     private final Set<String> hosts;

dialogue-core/src/main/java/com/palantir/dialogue/core/TimingEndpointChannel.java

@@ -31,12 +31,13 @@
 import com.palantir.logsafe.logger.SafeLoggerFactory;
 import com.palantir.tritium.metrics.registry.TaggedMetricRegistry;
 import java.io.IOException;
+import java.time.Duration;
 import java.util.concurrent.TimeUnit;
 
 final class TimingEndpointChannel implements EndpointChannel {
 
     private static final SafeLogger log = SafeLoggerFactory.get(TimingEndpointChannel.class);
-    private static final RateLimiter unknownThrowableLoggingRateLimiter = RateLimiter.create(1);
+    private static final RateLimiter unknownThrowableLoggingRateLimiter = RateLimiter.create(1, Duration.ZERO);
 
     private final EndpointChannel delegate;
     private final Timer successTimer;