Skip to content

Commit e306189

Browse files
committed
add manager warnings on client not ready
1 parent 6915880 commit e306189

File tree

1 file changed

+18
-0
lines changed

1 file changed

+18
-0
lines changed

splitio/client/manager.py

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,12 @@ def split_names(self):
3131
self._logger.error("Client has already been destroyed - no calls possible.")
3232
return []
3333

34+
if not self._factory.ready:
35+
self._logger.warning(
36+
"split_names: The SDK is not ready, results may be incorrect. "
37+
"Make sure to wait for SDK readiness before using this method"
38+
)
39+
3440
return self._storage.get_split_names()
3541

3642
def splits(self):
@@ -44,6 +50,12 @@ def splits(self):
4450
self._logger.error("Client has already been destroyed - no calls possible.")
4551
return []
4652

53+
if not self._factory.ready:
54+
self._logger.warning(
55+
"splits: The SDK is not ready, results may be incorrect. "
56+
"Make sure to wait for SDK readiness before using this method"
57+
)
58+
4759
return [split.to_split_view() for split in self._storage.get_all_splits()]
4860

4961
def split(self, feature_name):
@@ -66,6 +78,12 @@ def split(self, feature_name):
6678
self._storage
6779
)
6880

81+
if not self._factory.ready:
82+
self._logger.warning(
83+
"split: The SDK is not ready, results may be incorrect. "
84+
"Make sure to wait for SDK readiness before using this method"
85+
)
86+
6987
if feature_name is None:
7088
return None
7189

0 commit comments

Comments
 (0)