Enable AwsKms Client Caching by default to match KmsMasterKeyProvider

Author: WesleyRosenblum

src/main/java/com/amazonaws/encryptionsdk/kms/AwsKmsClientSupplier.java

@@ -89,7 +89,7 @@ class Builder {
         private ClientConfiguration clientConfiguration;
         private Set<String> allowedRegions = Collections.emptySet();
         private Set<String> excludedRegions = Collections.emptySet();
-        private boolean clientCachingEnabled = false;
+        private boolean clientCachingEnabled = true;
         private final Map<String, AWSKMS> clientsCache = new HashMap<>();
         private static final Set<String> AWSKMS_METHODS = new HashSet<>();
         private AWSKMSClientBuilder awsKmsClientBuilder;
@@ -188,7 +188,8 @@ public Builder excludedRegions(Set<String> regions) {
         }
 
         /**
-         * When set to true, allows for the AWSKMS client for each region to be cached and reused.
+         * When set to false, disables the AWSKMS client for each region from being cached and reused.
+         * By default, client caching is enabled.
          *
          * @param enabled Whether or not caching is enabled.
          */

src/test/java/com/amazonaws/encryptionsdk/kms/AwsKmsClientSupplierTest.java

@@ -128,7 +128,6 @@ void testClientCachingDisabled() {
     @Test
     void testClientCaching() {
         AwsKmsClientSupplier supplier = new AwsKmsClientSupplier.Builder(kmsClientBuilder)
-                .clientCaching(true)
                 .build();
 
         when(kmsClientBuilder.withRegion(REGION_1)).thenReturn(kmsClientBuilder);