Skip to content

Commit

Permalink
Fix bug
Browse files Browse the repository at this point in the history
Signed-off-by: Balaji Veeramani <[email protected]>
  • Loading branch information
bveeramani committed May 23, 2024
1 parent 29f6276 commit edb7ff7
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -128,7 +128,7 @@ def start(self, options: ExecutionOptions):
# upstream operators, leading to a spike in memory usage prior to steady state.
logger.debug(f"{self._name}: Waiting for {len(refs)} pool actors to start...")
try:
timeout = DataContext.get_current().wait_for_min_actors_timeout_s
timeout = DataContext.get_current().wait_for_min_actors_s
ray.get(refs, timeout=timeout)
except ray.exceptions.GetTimeoutError:
raise ray.exceptions.GetTimeoutError(
Expand Down
2 changes: 1 addition & 1 deletion python/ray/data/context.py
Original file line number Diff line number Diff line change
Expand Up @@ -275,7 +275,7 @@ class DataContext:
)
print_on_execution_start: bool = True
s3_try_create_dir: bool = DEFAULT_S3_TRY_CREATE_DIR
wait_for_min_actors_s: int = DEFAULT_WAIT_FOR_MIN_ACTORS_S
wait_for_min_actors_timeout_s: int = DEFAULT_WAIT_FOR_MIN_ACTORS_S

def __post_init__(self):
# The additonal ray remote args that should be added to
Expand Down
2 changes: 1 addition & 1 deletion python/ray/data/tests/test_actor_pool_map_operator.py
Original file line number Diff line number Diff line change
Expand Up @@ -525,7 +525,7 @@ def __call__(self, x):

from ray.exceptions import GetTimeoutError

ray.data.DataContext.get_current().wait_for_min_actors_timeout_s = 1
ray.data.DataContext.get_current().wait_for_min_actors_s = 1

with pytest.raises(
GetTimeoutError,
Expand Down

0 comments on commit edb7ff7

Please sign in to comment.