Skip to content

Commit 9be5a16

Browse files
authored
disable StreamingOutputBackpressurePolicy (#42000)
Cherrypick #41999 to 2.9.0. This can fix a regression for small-scale jobs (#41496). But it will also hurt stability for large-scale jobs. Signed-off-by: Hao Chen <chenh1024@gmail.com>
1 parent 15a558e commit 9be5a16

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

python/ray/data/_internal/execution/backpressure_policy/__init__.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,9 +11,8 @@
1111
# Default enabled backpressure policies and its config key.
1212
# Use `DataContext.set_config` to config it.
1313
# TODO(hchen): Enable ConcurrencyCapBackpressurePolicy by default.
14-
ENABLED_BACKPRESSURE_POLICIES = [
15-
StreamingOutputBackpressurePolicy,
16-
]
14+
# TODO(hchen): Enable StreamingOutputBackpressurePolicy by default.
15+
ENABLED_BACKPRESSURE_POLICIES = []
1716
ENABLED_BACKPRESSURE_POLICIES_CONFIG_KEY = "backpressure_policies.enabled"
1817

1918

python/ray/data/tests/test_backpressure_policies.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
from unittest.mock import MagicMock, patch
77

88
import numpy as np
9+
import pytest
910

1011
import ray
1112
from ray.data._internal.execution.backpressure_policy import (
@@ -409,6 +410,7 @@ def foo(self):
409410
)
410411

411412

413+
@pytest.mark.skip("Re-enable after enabling StreamingOutputBackpressurePolicy.")
412414
def test_large_e2e_backpressure(shutdown_only, restore_data_context): # noqa: F811
413415
"""Test backpressure on a synthetic large-scale workload."""
414416
# The cluster has 10 CPUs and 200MB object store memory.

0 commit comments

Comments
 (0)