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

svc-excavator-bot · web-flow · commit fbd3163e0977 · 2025-06-12T19:51:30.000Z
diff --git a/build.gradle b/build.gradle
@@ -14,7 +14,7 @@ buildscript {
         classpath 'com.palantir.javaformat:gradle-palantir-java-format:2.68.0'
         classpath 'com.palantir.suppressible-error-prone:gradle-suppressible-error-prone:2.9.0'
         classpath 'com.palantir.gradle.revapi:gradle-revapi:1.8.0'
-        classpath 'com.palantir.baseline:gradle-baseline-java:6.34.0'
+        classpath 'com.palantir.baseline:gradle-baseline-java:6.35.0'
         classpath 'com.palantir.gradle.consistentversions:gradle-consistent-versions:2.34.0'
     }
 }
diff --git a/errors/src/main/java/com/palantir/conjure/java/api/errors/QosException.java b/errors/src/main/java/com/palantir/conjure/java/api/errors/QosException.java
@@ -224,7 +224,6 @@ public String getLogMessage() {
             return "Suggested request throttling";
         }
 
-        @SuppressWarnings("for-rollout:Java8ApiChecker")
         @Override
         public List<Arg<?>> getArgs() {
             return List.of(
@@ -276,7 +275,6 @@ public String getLogMessage() {
             return this.getClass().getSimpleName() + ": Requesting retry";
         }
 
-        @SuppressWarnings("for-rollout:Java8ApiChecker")
         @Unsafe
         @Override
         public List<Arg<?>> getArgs() {
@@ -320,7 +318,6 @@ public String getLogMessage() {
             return SERVER_UNAVAILABLE;
         }
 
-        @SuppressWarnings("for-rollout:Java8ApiChecker")
         @Override
         public List<Arg<?>> getArgs() {
             return List.of(
diff --git a/errors/src/main/java/com/palantir/conjure/java/api/errors/QosReason.java b/errors/src/main/java/com/palantir/conjure/java/api/errors/QosReason.java
@@ -53,11 +53,7 @@ private QosReason(@Safe String reason, Optional<RetryHint> retryHint, Optional<D
     }
 
     public static QosReason of(
-            @SuppressWarnings("for-rollout:UnnecessarilyFullyQualified")
-                    @Safe
-                    @CompileTimeConstant
-                    @org.intellij.lang.annotations.Pattern(PATTERN_STRING)
-                    String reason) {
+            @Safe @CompileTimeConstant @org.intellij.lang.annotations.Pattern(PATTERN_STRING) String reason) {
         return new QosReason(reason, Optional.empty(), Optional.empty());
     }
 
@@ -121,11 +117,7 @@ public Builder from(QosReason other) {
         }
 
         public Builder reason(
-                @SuppressWarnings("for-rollout:UnnecessarilyFullyQualified")
-                        @Safe
-                        @CompileTimeConstant
-                        @org.intellij.lang.annotations.Pattern(PATTERN_STRING)
-                        String value) {
+                @Safe @CompileTimeConstant @org.intellij.lang.annotations.Pattern(PATTERN_STRING) String value) {
             this.reason = Preconditions.checkNotNull(value, "reason");
             return this;
         }
diff --git a/errors/src/main/java/com/palantir/conjure/java/api/errors/QosReasons.java b/errors/src/main/java/com/palantir/conjure/java/api/errors/QosReasons.java
@@ -42,7 +42,6 @@ public static <T> void encodeToResponse(
         }
     }
 
-    @SuppressWarnings("for-rollout:Java8ApiChecker")
     public static <T> QosReason parseFromResponse(T response, QosResponseDecodingAdapter<? super T> adapter) {
         Optional<String> maybeDueTo = adapter.getFirstHeader(response, DUE_TO_HEADER);
         Optional<String> maybeRetryHint = adapter.getFirstHeader(response, RETRY_HINT_HEADER);
diff --git a/errors/src/main/java/com/palantir/conjure/java/api/errors/RemoteException.java b/errors/src/main/java/com/palantir/conjure/java/api/errors/RemoteException.java
@@ -63,7 +63,6 @@ public RemoteException(SerializableError error, int status) {
     @Override
     public String getMessage() {
         // This field is not used in most environments so the cost of computation may be avoided.
-        @SuppressWarnings("for-rollout:Var")
         String messageValue = unsafeMessage;
         if (messageValue == null) {
             messageValue = renderUnsafeMessage();
diff --git a/errors/src/main/java/com/palantir/conjure/java/api/errors/ServiceExceptionUtils.java b/errors/src/main/java/com/palantir/conjure/java/api/errors/ServiceExceptionUtils.java
@@ -70,7 +70,6 @@ static String renderUnsafeMessage(String exceptionName, ErrorType errorType, Arg
 
         StringBuilder builder = new StringBuilder();
         builder.append(message).append(": {");
-        @SuppressWarnings("for-rollout:Var")
         boolean first = true;
         for (Arg<?> arg : args) {
             if (arg == null) {
diff --git a/errors/src/main/java/com/palantir/conjure/java/api/errors/UnknownRemoteException.java b/errors/src/main/java/com/palantir/conjure/java/api/errors/UnknownRemoteException.java
@@ -52,7 +52,6 @@ public String getLogMessage() {
         return getMessage();
     }
 
-    @SuppressWarnings("for-rollout:Java8ApiChecker")
     @Unsafe
     @Override
     public List<Arg<?>> getArgs() {
diff --git a/errors/src/test/java/com/palantir/conjure/java/api/errors/ErrorTypeTest.java b/errors/src/test/java/com/palantir/conjure/java/api/errors/ErrorTypeTest.java
@@ -26,7 +26,6 @@
 
 public final class ErrorTypeTest {
 
-    @SuppressWarnings("for-rollout:DifferentNameButSame")
     @Test
     public void testNameMustBeCamelCaseWithOptionalNameSpace() throws Exception {
         String[] badNames = new String[] {":", "foo:Bar", ":Bar", "Bar:", "foo:bar", "Foo:bar", "Foo:2Bar"};
@@ -65,26 +64,21 @@ public void testDefaultErrorTypeHttpErrorCodes() throws Exception {
         assertThat(ErrorType.INTERNAL.httpErrorCode()).isEqualTo(500);
     }
 
-    @SuppressWarnings("for-rollout:DifferentNameButSame")
     @Test
     public void testCanCreateCustomClientAndServerErrors() throws Exception {
-        @SuppressWarnings("for-rollout:DifferentNameButSame")
         ErrorType customClient = ErrorType.create(ErrorType.Code.CUSTOM_CLIENT, "Namespace:MyDesc");
         assertThat(customClient.code()).isEqualTo(ErrorType.Code.CUSTOM_CLIENT);
         assertThat(customClient.httpErrorCode()).isEqualTo(400);
         assertThat(customClient.name()).isEqualTo("Namespace:MyDesc");
 
-        @SuppressWarnings("for-rollout:DifferentNameButSame")
         ErrorType customServer = ErrorType.create(ErrorType.Code.CUSTOM_SERVER, "Namespace:MyDesc");
         assertThat(customServer.code()).isEqualTo(ErrorType.Code.CUSTOM_SERVER);
         assertThat(customServer.httpErrorCode()).isEqualTo(500);
         assertThat(customServer.name()).isEqualTo("Namespace:MyDesc");
     }
 
-    @SuppressWarnings("for-rollout:DifferentNameButSame")
     @Test
     public void testCanCreateNewErrorTypes() throws Exception {
-        @SuppressWarnings("for-rollout:DifferentNameButSame")
         ErrorType error = ErrorType.create(ErrorType.Code.FAILED_PRECONDITION, "Namespace:MyDesc");
         assertThat(error.code()).isEqualTo(ErrorType.Code.FAILED_PRECONDITION);
         assertThat(error.httpErrorCode()).isEqualTo(500);
diff --git a/errors/src/test/java/com/palantir/conjure/java/api/errors/RemoteExceptionTest.java b/errors/src/test/java/com/palantir/conjure/java/api/errors/RemoteExceptionTest.java
@@ -27,15 +27,12 @@ public final class RemoteExceptionTest {
     @Test
     public void testJavaSerialization() {
         // With explicit error instance
-        @SuppressWarnings("for-rollout:Var")
         SerializableError error = new SerializableError.Builder()
                 .errorCode("errorCode")
                 .errorName("errorName")
                 .errorInstanceId("errorId")
                 .build();
-        @SuppressWarnings("for-rollout:Var")
         RemoteException expected = new RemoteException(error, 500);
-        @SuppressWarnings("for-rollout:Var")
         RemoteException actual = SerializationUtils.deserialize(SerializationUtils.serialize(expected));
         assertThat(actual).isEqualToComparingFieldByField(expected);
 
diff --git a/errors/src/test/java/com/palantir/conjure/java/api/errors/SerializableErrorTest.java b/errors/src/test/java/com/palantir/conjure/java/api/errors/SerializableErrorTest.java
@@ -75,7 +75,6 @@ public void forException_arg_key_collisions_just_use_the_last_one() {
     @Test
     public void forException_listArgValue_serializesWithToString() {
         ErrorType error = ErrorType.INTERNAL;
-        @SuppressWarnings("for-rollout:Java8ApiChecker")
         ServiceException exception = new ServiceException(
                 error, SafeArg.of("safe-list", List.of("1", "2")), UnsafeArg.of("unsafe-list", List.of("3", "4")));
 
@@ -92,7 +91,6 @@ public void forException_listArgValue_serializesWithToString() {
     @Test
     public void forException_mapArgValue_serializesWithToString() {
         ErrorType error = ErrorType.INTERNAL;
-        @SuppressWarnings("for-rollout:Java8ApiChecker")
         ServiceException exception = new ServiceException(
                 error, SafeArg.of("safe-map", Map.of("1", "2")), UnsafeArg.of("unsafe-map", Map.of("ABC", "DEF")));
 
diff --git a/extras/jackson-support/src/main/java/com/palantir/conjure/java/api/ext/jackson/PathDeserializer.java b/extras/jackson-support/src/main/java/com/palantir/conjure/java/api/ext/jackson/PathDeserializer.java
@@ -62,7 +62,6 @@ public String getLogMessage() {
             return logMessage;
         }
 
-        @SuppressWarnings("for-rollout:Java8ApiChecker")
         @Override
         public List<Arg<?>> getArgs() {
             return List.of();
diff --git a/extras/jackson-support/src/main/java/com/palantir/conjure/java/api/ext/jackson/ShimJdk7Module.java b/extras/jackson-support/src/main/java/com/palantir/conjure/java/api/ext/jackson/ShimJdk7Module.java
@@ -27,7 +27,6 @@ public final class ShimJdk7Module extends SimpleModule {
     public ShimJdk7Module() {
         super(ShimJdk7Module.class.getCanonicalName());
 
-        @SuppressWarnings({"for-rollout:UnnecessaryFinal", "for-rollout:Var"})
         final JsonSerializer<Object> stringSer = ToStringSerializer.instance;
         addSerializer(Path.class, stringSer);
         addDeserializer(Path.class, new PathDeserializer());
diff --git a/service-config/src/main/java/com/palantir/conjure/java/api/config/service/HostAndPort.java b/service-config/src/main/java/com/palantir/conjure/java/api/config/service/HostAndPort.java
@@ -61,7 +61,6 @@ boolean hasPort() {
     public static HostAndPort fromString(String hostPortString) {
         Preconditions.checkNotNull(hostPortString, "hostPortString");
         String host;
-        @SuppressWarnings("for-rollout:Var")
         String portString = null;
 
         if (hostPortString.startsWith("[")) {
@@ -80,7 +79,6 @@ public static HostAndPort fromString(String hostPortString) {
             }
         }
 
-        @SuppressWarnings("for-rollout:Var")
         int port = NO_PORT;
         if (portString != null && portString.length() > 0) {
             // Try to parse the whole port string as a number.
diff --git a/service-config/src/main/java/com/palantir/conjure/java/api/config/service/HumanReadableDuration.java b/service-config/src/main/java/com/palantir/conjure/java/api/config/service/HumanReadableDuration.java
@@ -95,13 +95,10 @@ public static HumanReadableDuration days(long count) {
 
     @JsonCreator
     public static HumanReadableDuration valueOf(String duration) {
-        @SuppressWarnings({"for-rollout:UnnecessaryFinal", "for-rollout:Var"})
         final Matcher matcher = DURATION_PATTERN.matcher(duration);
         Preconditions.checkArgument(matcher.matches(), "Invalid duration", SafeArg.of("duration", duration));
 
-        @SuppressWarnings({"for-rollout:UnnecessaryFinal", "for-rollout:Var"})
         final long count = Long.parseLong(matcher.group(1));
-        @SuppressWarnings({"for-rollout:UnnecessaryFinal", "for-rollout:Var"})
         final TimeUnit unit = SUFFIXES.get(matcher.group(2));
         if (unit == null) {
             throw new SafeIllegalArgumentException("Invalid duration", SafeArg.of("duration", duration));
@@ -198,7 +195,6 @@ public boolean equals(Object obj) {
         if ((obj == null) || (getClass() != obj.getClass())) {
             return false;
         }
-        @SuppressWarnings({"for-rollout:UnnecessaryFinal", "for-rollout:Var"})
         final HumanReadableDuration duration = (HumanReadableDuration) obj;
         return (count == duration.count) && (unit == duration.unit);
     }
@@ -211,7 +207,6 @@ public int hashCode() {
     @Override
     @JsonValue
     public String toString() {
-        @SuppressWarnings("for-rollout:Var")
         String units = unit.toString().toLowerCase(Locale.ENGLISH);
         if (count == 1) {
             units = units.substring(0, units.length() - 1);
diff --git a/service-config/src/main/java/com/palantir/conjure/java/api/config/service/ProxyConfiguration.java b/service-config/src/main/java/com/palantir/conjure/java/api/config/service/ProxyConfiguration.java
@@ -95,11 +95,7 @@ public Type type() {
         return Type.HTTP;
     }
 
-    @SuppressWarnings({
-        "for-rollout:Java8ApiChecker",
-        "for-rollout:StatementSwitchToExpressionSwitch",
-        "for-rollout:UnnecessaryDefaultInEnumSwitch"
-    })
+    @SuppressWarnings({"for-rollout:StatementSwitchToExpressionSwitch", "for-rollout:UnnecessaryDefaultInEnumSwitch"})
     @Value.Check
     protected final void check() {
         switch (type()) {
diff --git a/service-config/src/main/java/com/palantir/conjure/java/api/config/service/UserAgents.java b/service-config/src/main/java/com/palantir/conjure/java/api/config/service/UserAgents.java
@@ -125,7 +125,6 @@ public static String tryParsePrimaryName(@Safe String userAgent) {
         // it will possibly ignore leading characters (even non-whitespace) up until the "foo/1.2.3" part
         // so we further split primaryName into _just_ the set of characters immediately preceding
         // the "/" for which isValidCharForName() returns true.
-        @SuppressWarnings("for-rollout:Var")
         int lindex = split - 1;
         while (lindex >= 0) {
             if (!isValidCharForName(userAgent.charAt(lindex))) {
@@ -152,7 +151,6 @@ private static UserAgent parseInternal(@Safe String userAgent, boolean lenient)
         ImmutableUserAgent.Builder builder = ImmutableUserAgent.builder();
 
         Matcher matcher = SEGMENT_PATTERN.matcher(userAgent);
-        @SuppressWarnings("for-rollout:Var")
         boolean foundFirst = false;
         while (matcher.find()) {
             String name = matcher.group(1);
@@ -238,7 +236,6 @@ static boolean isValidName(String name) {
         if (name == null || name.isEmpty()) {
             return false;
         }
-        @SuppressWarnings("for-rollout:Var")
         char ch = name.charAt(0);
         if (!isAlpha(ch)) {
             return false;
diff --git a/service-config/src/main/java/com/palantir/conjure/java/api/config/service/VersionParser.java b/service-config/src/main/java/com/palantir/conjure/java/api/config/service/VersionParser.java
@@ -39,7 +39,6 @@ private VersionParser() {}
 
     /** Returns the count of version number groups parsed if a valid version string, or -1 otherwise. */
     public static int countNumericDotGroups(String string) {
-        @SuppressWarnings("for-rollout:Var")
         long state = 0;
         for (int i = 1; getIndex(state) < string.length(); i++) {
             state = number(string, getIndex(state));
@@ -64,7 +63,6 @@ public static int countNumericDotGroups(String string) {
     private static final int PARSE_FAILED = Integer.MIN_VALUE;
 
     static long number(String string, int startIndex) {
-        @SuppressWarnings("for-rollout:Var")
         int next = startIndex;
         int len = string.length();
         while (next < len) {
@@ -81,7 +79,6 @@ static long number(String string, int startIndex) {
             return ok(next, Character.digit(string.codePointAt(startIndex), 10));
         } else {
             try {
-                @SuppressWarnings("for-rollout:Java8ApiChecker")
                 int result = Integer.parseUnsignedInt(string, startIndex, next, 10);
                 if (result < 0) {
                     // i.e. we overflowed the int
diff --git a/service-config/src/test/java/com/palantir/conjure/java/api/config/service/UserAgentTest.java b/service-config/src/test/java/com/palantir/conjure/java/api/config/service/UserAgentTest.java
@@ -32,7 +32,6 @@
 
 public class UserAgentTest {
 
-    @SuppressWarnings("for-rollout:DifferentNameButSame")
     @Test
     public void validAndInvalidNodeSyntax() {
         // Valid nodeId
@@ -52,30 +51,25 @@ public void validAndInvalidNodeSyntax() {
 
     @Test
     public void testCorrectHeaderFormatWithNodeId() {
-        @SuppressWarnings("for-rollout:DifferentNameButSame")
         UserAgent baseUserAgent = UserAgent.of(UserAgent.Agent.of("service", "1.0.0"), "myNode");
         assertThat(UserAgents.format(baseUserAgent)).isEqualTo("service/1.0.0 (nodeId:myNode)");
 
-        @SuppressWarnings("for-rollout:DifferentNameButSame")
         UserAgent derivedAgent = baseUserAgent.addAgent(UserAgent.Agent.of("conjure", "2.0.0"));
         assertThat(UserAgents.format(derivedAgent)).isEqualTo("service/1.0.0 (nodeId:myNode) conjure/2.0.0");
     }
 
     @Test
     public void testCorrectHeaderFormatWithoutNodeId() {
-        @SuppressWarnings("for-rollout:DifferentNameButSame")
         UserAgent baseUserAgent = UserAgent.of(UserAgent.Agent.of("service", "1.0.0"));
         assertThat(UserAgents.format(baseUserAgent)).isEqualTo("service/1.0.0");
 
-        @SuppressWarnings("for-rollout:DifferentNameButSame")
         UserAgent derivedAgent = baseUserAgent.addAgent(UserAgent.Agent.of("conjure", "2.0.0"));
         assertThat(UserAgents.format(derivedAgent)).isEqualTo("service/1.0.0 conjure/2.0.0");
     }
 
     @Test
     void testPrimaryWithInformational() {
         UserAgent baseUserAgent = UserAgent.of(Agent.of("service", "1.0.0"));
-        @SuppressWarnings("for-rollout:PreferredInterfaceType")
         List<Agent> info = ImmutableList.of(Agent.of("conjure", "1.2.3"), Agent.of("jdk", "17.0.4.1"));
         UserAgent first = UserAgent.of(baseUserAgent, info);
         assertThat(first).satisfies(agent -> {
@@ -86,7 +80,6 @@ void testPrimaryWithInformational() {
             assertThat(UserAgent.of(agent, ImmutableList.of())).isEqualTo(agent);
         });
 
-        @SuppressWarnings("for-rollout:PreferredInterfaceType")
         List<Agent> moreInfo = ImmutableList.of(Agent.of("test", "4.5.6"));
         assertThat(UserAgent.of(first, moreInfo)).satisfies(agent -> {
             assertThat(agent.primary()).isEqualTo(baseUserAgent.primary());
@@ -97,7 +90,6 @@ void testPrimaryWithInformational() {
         });
     }
 
-    @SuppressWarnings("for-rollout:DifferentNameButSame")
     @Test
     public void testInvalidServiceName() {
         assertThatLoggableExceptionThrownBy(() -> UserAgent.Agent.of("invalid service name", "1.0.0"))
@@ -106,7 +98,6 @@ public void testInvalidServiceName() {
                 .isInstanceOf(IllegalArgumentException.class);
     }
 
-    @SuppressWarnings("for-rollout:DifferentNameButSame")
     @Test
     public void testInvalidNodeId() {
         assertThatLoggableExceptionThrownBy(
@@ -116,7 +107,6 @@ public void testInvalidNodeId() {
                 .hasLogMessage("Illegal node id format");
     }
 
-    @SuppressWarnings("for-rollout:DifferentNameButSame")
     @Test
     public void testInvalidVersion() {
         assertThat(UserAgents.format(UserAgent.of(UserAgent.Agent.of("serviceName", "1 0 0"), "myNode")))

Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,7 @@ buildscript {`
`14`	`14`	`classpath 'com.palantir.javaformat:gradle-palantir-java-format:2.68.0'`
`15`	`15`	`classpath 'com.palantir.suppressible-error-prone:gradle-suppressible-error-prone:2.9.0'`
`16`	`16`	`classpath 'com.palantir.gradle.revapi:gradle-revapi:1.8.0'`
`17`		`- classpath 'com.palantir.baseline:gradle-baseline-java:6.34.0'`
	`17`	`+ classpath 'com.palantir.baseline:gradle-baseline-java:6.35.0'`
`18`	`18`	`classpath 'com.palantir.gradle.consistentversions:gradle-consistent-versions:2.34.0'`
`19`	`19`	`}`
`20`	`20`	`}`
Original file line number	Diff line number	Diff line change
`@@ -224,7 +224,6 @@ public String getLogMessage() {`
`224`	`224`	`return "Suggested request throttling";`
`225`	`225`	`}`
`226`	`226`
`227`		`- @SuppressWarnings("for-rollout:Java8ApiChecker")`
`228`	`227`	`@Override`
`229`	`228`	`public List<Arg<?>> getArgs() {`
`230`	`229`	`return List.of(`
`@@ -276,7 +275,6 @@ public String getLogMessage() {`
`276`	`275`	`return this.getClass().getSimpleName() + ": Requesting retry";`
`277`	`276`	`}`
`278`	`277`
`279`		`- @SuppressWarnings("for-rollout:Java8ApiChecker")`
`280`	`278`	`@Unsafe`
`281`	`279`	`@Override`
`282`	`280`	`public List<Arg<?>> getArgs() {`
`@@ -320,7 +318,6 @@ public String getLogMessage() {`
`320`	`318`	`return SERVER_UNAVAILABLE;`
`321`	`319`	`}`
`322`	`320`
`323`		`- @SuppressWarnings("for-rollout:Java8ApiChecker")`
`324`	`321`	`@Override`
`325`	`322`	`public List<Arg<?>> getArgs() {`
`326`	`323`	`return List.of(`
Original file line number	Diff line number	Diff line change
`@@ -53,11 +53,7 @@ private QosReason(@Safe String reason, Optional<RetryHint> retryHint, Optional<D`
`53`	`53`	`}`
`54`	`54`
`55`	`55`	`public static QosReason of(`
`56`		`- @SuppressWarnings("for-rollout:UnnecessarilyFullyQualified")`
`57`		`- @Safe`
`58`		`- @CompileTimeConstant`
`59`		`- @org.intellij.lang.annotations.Pattern(PATTERN_STRING)`
`60`		`- String reason) {`
	`56`	`+ @Safe @CompileTimeConstant @org.intellij.lang.annotations.Pattern(PATTERN_STRING) String reason) {`
`61`	`57`	`return new QosReason(reason, Optional.empty(), Optional.empty());`
`62`	`58`	`}`
`63`	`59`
`@@ -121,11 +117,7 @@ public Builder from(QosReason other) {`
`121`	`117`	`}`
`122`	`118`
`123`	`119`	`public Builder reason(`
`124`		`- @SuppressWarnings("for-rollout:UnnecessarilyFullyQualified")`
`125`		`- @Safe`
`126`		`- @CompileTimeConstant`
`127`		`- @org.intellij.lang.annotations.Pattern(PATTERN_STRING)`
`128`		`- String value) {`
	`120`	`+ @Safe @CompileTimeConstant @org.intellij.lang.annotations.Pattern(PATTERN_STRING) String value) {`
`129`	`121`	`this.reason = Preconditions.checkNotNull(value, "reason");`
`130`	`122`	`return this;`
`131`	`123`	`}`
Original file line number	Diff line number	Diff line change
`@@ -42,7 +42,6 @@ public static <T> void encodeToResponse(`
`42`	`42`	`}`
`43`	`43`	`}`
`44`	`44`
`45`		`- @SuppressWarnings("for-rollout:Java8ApiChecker")`
`46`	`45`	`public static <T> QosReason parseFromResponse(T response, QosResponseDecodingAdapter<? super T> adapter) {`
`47`	`46`	`Optional<String> maybeDueTo = adapter.getFirstHeader(response, DUE_TO_HEADER);`
`48`	`47`	`Optional<String> maybeRetryHint = adapter.getFirstHeader(response, RETRY_HINT_HEADER);`
Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,6 @@ public String getLogMessage() {`
`52`	`52`	`return getMessage();`
`53`	`53`	`}`
`54`	`54`
`55`		`- @SuppressWarnings("for-rollout:Java8ApiChecker")`
`56`	`55`	`@Unsafe`
`57`	`56`	`@Override`
`58`	`57`	`public List<Arg<?>> getArgs() {`