Addressing Claude-Code review feedback

Author: nhachicha

driver-core/src/main/com/mongodb/internal/connection/BackpressureErrorLabeler.java

@@ -17,7 +17,6 @@
 package com.mongodb.internal.connection;
 
 import com.mongodb.MongoException;
-import com.mongodb.MongoSecurityException;
 import com.mongodb.MongoSocketException;
 
 import javax.net.ssl.SSLHandshakeException;
@@ -43,12 +42,6 @@ private BackpressureErrorLabeler() {
     }
 
     static void applyLabelsIfEligible(final Throwable t) {
-        if (!(t instanceof MongoException)) {
-            return;
-        }
-        if (t instanceof MongoSecurityException) {
-            return;
-        }
         if (!(t instanceof MongoSocketException)) {
             return;
         }
@@ -66,7 +59,7 @@ static void applyLabelsIfEligible(final Throwable t) {
         mongoException.addLabel(MongoException.RETRYABLE_ERROR_LABEL);
     }
 
-    static boolean isDnsLookupFailure(final Throwable t) {
+    private static boolean isDnsLookupFailure(final Throwable t) {
         Throwable cause = t.getCause();
         while (cause != null) {
             if (cause instanceof UnknownHostException) {
@@ -77,10 +70,7 @@ static boolean isDnsLookupFailure(final Throwable t) {
         return false;
     }
 
-    static boolean isTlsConfigurationError(final Throwable t) {
-        if (!(t instanceof MongoSocketException)) {
-            return false;
-        }
+    private static boolean isTlsConfigurationError(final Throwable t) {
         Throwable cause = t.getCause();
         while (cause != null) {
             if (cause instanceof CertificateException

driver-core/src/test/unit/com/mongodb/internal/connection/DefaultServerSpecification.groovy

@@ -259,7 +259,7 @@ class DefaultServerSpecification extends Specification {
         ]
     }
 
-    def 'DNS lookup failure should invalidate the pool'() {
+    def 'DNS lookup failure should not be labeled as backpressure and should invalidate the pool'() {
         given:
         def exceptionToThrow = new MongoSocketException('DNS lookup failed', new ServerAddress(),
                 new UnknownHostException('no such host'))

driver-sync/src/test/functional/com/mongodb/client/ServerDiscoveryAndMonitoringProseTests.java

@@ -327,7 +327,11 @@ public void testConnectionPoolBackpressure() throws InterruptedException {
                         failedCheckOutCount >= 10);
                 assertEquals(0, connectionPoolListener.countEvents(ConnectionPoolClearedEvent.class));
             } finally {
-                Thread.sleep(1000);
+                try {
+                    Thread.sleep(1000);
+                } catch (InterruptedException e) {
+                    Thread.currentThread().interrupt();
+                }
                 adminDatabase.runCommand(new Document("setParameter", 1)
                         .append("ingressConnectionEstablishmentRateLimiterEnabled", false));
             }