Merge pull request #453 from splitio/SDKS-7626

[SDKS-7626] Add feature flag name in ready warning log

Author: nmayorsplit

client/src/main/java/io/split/client/SplitClientImpl.java

@@ -23,6 +23,7 @@
 import org.slf4j.LoggerFactory;
 
 import java.util.ArrayList;
+import java.util.Arrays;
 import java.util.Collections;
 import java.util.HashMap;
 import java.util.List;
@@ -226,7 +227,7 @@ private SplitResult getTreatmentWithConfigInternal(String matchingKey, String bu
                                                        Object> attributes, MethodEnum methodEnum) {
         long initTime = System.currentTimeMillis();
         try {
-            checkSDKReady(methodEnum);
+            checkSDKReady(methodEnum, Arrays.asList(featureFlag));
 
             if (_container.isDestroyed()) {
                 _log.error("Client has already been destroyed - no calls possible");
@@ -290,7 +291,7 @@ private Map<String, SplitResult> getTreatmentsWithConfigInternal(String matching
             return new HashMap<>();
         }
         try {
-            checkSDKReady(methodEnum);
+            checkSDKReady(methodEnum, featureFlagNames);
             if (_container.isDestroyed()) {
                 _log.error("Client has already been destroyed - no calls possible");
                 return createMapControl(featureFlagNames);
@@ -360,10 +361,11 @@ private Event createEvent(String key, String trafficType, String eventType) {
         return event;
     }
 
-    private void checkSDKReady(MethodEnum methodEnum) {
+    private void checkSDKReady(MethodEnum methodEnum, List<String> featureFlagNames) {
+        String toPrint =  featureFlagNames.size() == 1 ? featureFlagNames.get(0): String.join(",", featureFlagNames);
         if (!_gates.isSDKReady()) {
-            _log.warn(String.format("%s: the SDK is not ready, results may be incorrect. Make sure to wait for SDK readiness " +
-                            "before using this method", methodEnum.getMethod()));
+            _log.warn(String.format("%s: the SDK is not ready, results may be incorrect for feature flag %s. Make sure to wait for " +
+                            "SDK readiness before using this method", methodEnum.getMethod(), toPrint));
             _telemetryConfigProducer.recordNonReadyUsage();
         }
     }

client/src/main/java/io/split/client/SplitManagerImpl.java

@@ -58,7 +58,8 @@ public List<SplitView> splits() {
     @Override
     public SplitView split(String featureFlagName) {
         if (!_gates.isSDKReady()) { {
-            _log.warn("split: the SDK is not ready, results may be incorrect. Make sure to wait for SDK readiness before using this method");
+            _log.warn(String.format("the SDK is not ready, results may be incorrect for feature flag %s. Make sure to wait " +
+                    "for SDK readiness before using this method", featureFlagName));
             _telemetryConfigProducer.recordNonReadyUsage();
         }}
         Optional<String> result = SplitNameValidator.isValid(featureFlagName, "split");