Skip to content

Commit 98e4093

Browse files
authored
Merge pull request #155 from splitio/cleanup/tasks_init
simplify task initialization logic
2 parents c862382 + 9429b3c commit 98e4093

File tree

2 files changed

+4
-8
lines changed

2 files changed

+4
-8
lines changed

splitio/tasks/segment_sync.py

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -81,10 +81,8 @@ def _ensure_segment_is_updated(self, segment_name):
8181
:param segment_name: Name of the segment to update.
8282
:type segment_name: str
8383
"""
84-
while True:
85-
ready = self._update_segment(segment_name)
86-
if ready:
87-
break
84+
while not self._update_segment(segment_name):
85+
pass
8886

8987
def start(self):
9088
"""Start segment synchronization."""

splitio/tasks/split_sync.py

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -56,10 +56,8 @@ def _update_splits(self):
5656

5757
def _on_start(self):
5858
"""Wait until splits are in sync and set the flag to true."""
59-
while True:
60-
ready = self._update_splits()
61-
if ready:
62-
break
59+
while not self._update_splits():
60+
pass
6361

6462
self._ready_flag.set()
6563
return True

0 commit comments

Comments
 (0)