UrlConnectionSdkHttpService should have higher priorty than AwsCrtSdkHttpService

Author: zoewangg

core/sdk-core/src/main/java/software/amazon/awssdk/core/internal/http/loader/ClasspathSdkHttpServiceProvider.java

@@ -39,8 +39,8 @@ final class ClasspathSdkHttpServiceProvider<T> implements SdkHttpServiceProvider
     static final Map<String, Integer> SYNC_HTTP_SERVICES_PRIORITY =
         ImmutableMap.<String, Integer>builder()
                     .put("software.amazon.awssdk.http.apache.ApacheSdkHttpService", 1)
-                    .put("software.amazon.awssdk.http.crt.AwsCrtSdkHttpService", 2)
-                    .put("software.amazon.awssdk.http.urlconnection.UrlConnectionSdkHttpService", 3)
+                    .put("software.amazon.awssdk.http.urlconnection.UrlConnectionSdkHttpService", 2)
+                    .put("software.amazon.awssdk.http.crt.AwsCrtSdkHttpService", 3)
                     .build();
 
     static final Map<String, Integer> ASYNC_HTTP_SERVICES_PRIORITY =
@@ -94,7 +94,7 @@ private String logServices(Queue<T> impls) {
         String message = count == 1 ? "The HTTP implementation loaded is " + impl :
                          String.format(
                              "Multiple HTTP implementations were found on the classpath. The SDK will use %s since it has the "
-                             + "highest. The multiple implementations found were: %s",
+                             + "highest priority. The multiple implementations found were: %s",
                              impl,
                              implText);
 

test/stability-tests/src/it/java/software/amazon/awssdk/stability/tests/s3/S3BaseStabilityTest.java

@@ -58,7 +58,7 @@ public abstract class S3BaseStabilityTest extends AwsTestBase {
 
 
     // The JVM does a bunch under the hood, so leave some room for magic.
-    private static final int ALLOWED_THREAD_OVERHEAD = 20;
+    private static final int ALLOWED_THREAD_OVERHEAD = 50;
 
     protected int allowedPeakThreads;