update: modify DefaultCmabClient constructor to use default retry configuration

Author: FarhanAnjum-opti

core-httpclient-impl/src/main/java/com/optimizely/ab/OptimizelyFactory.java

@@ -23,7 +23,6 @@
 import org.slf4j.LoggerFactory;
 
 import com.optimizely.ab.cmab.DefaultCmabClient;
-import com.optimizely.ab.cmab.client.CmabClientConfig;
 import com.optimizely.ab.cmab.service.CmabService;
 import com.optimizely.ab.cmab.service.DefaultCmabService;
 import com.optimizely.ab.config.HttpProjectConfigManager;
@@ -390,7 +389,7 @@ public static Optimizely newDefaultInstance(ProjectConfigManager configManager,
 
         // If no cmabService provided, create default one
         if (cmabService == null) {
-            DefaultCmabClient defaultCmabClient = new DefaultCmabClient(CmabClientConfig.withDefaultRetry());
+            DefaultCmabClient defaultCmabClient = new DefaultCmabClient();
             cmabService = DefaultCmabService.builder()
                 .withClient(defaultCmabClient)
                 .build();

core-httpclient-impl/src/main/java/com/optimizely/ab/cmab/DefaultCmabClient.java

@@ -62,9 +62,9 @@ public DefaultCmabClient(CmabClientConfig config) {
         this(null, config);
     }
 
-    // Default constructor (no retry, default HTTP client)
+    // Default constructor (default HTTP client, default retry config)
     public DefaultCmabClient() {
-        this(null, CmabClientConfig.withNoRetry());
+        this(null, CmabClientConfig.withDefaultRetry());
     }
 
     // Extract HTTP client creation logic