Skip to content

[Data] Fail map_batches benchmark on worker OOM#63474

Merged
bveeramani merged 1 commit into
masterfrom
map-batches-fail-on-worker-oom
May 19, 2026
Merged

[Data] Fail map_batches benchmark on worker OOM#63474
bveeramani merged 1 commit into
masterfrom
map-batches-fail-on-worker-oom

Conversation

@bveeramani

@bveeramani bveeramani commented May 18, 2026

Copy link
Copy Markdown
Member

Description

The map_batches release benchmark had RAYTEST_FAIL_ON_WORKER_OOM=0. After we landed some changes to minimize memory bloat like #62302, the test no longer OOMs, so I'm re-enabling the flag.

Related issues

None

Signed-off-by: Balaji Veeramani <bveeramani@berkeley.edu>
@gemini-code-assist

Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@ayushk7102 ayushk7102 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@bveeramani bveeramani enabled auto-merge (squash) May 19, 2026 00:04
@github-actions github-actions Bot added the go add ONLY when ready to merge, run all tests label May 19, 2026
@bveeramani bveeramani merged commit fcf99d3 into master May 19, 2026
6 of 7 checks passed
@bveeramani bveeramani deleted the map-batches-fail-on-worker-oom branch May 19, 2026 00:26
TruongQuangPhat pushed a commit to cyhapun/ray-fix-issue that referenced this pull request May 27, 2026
## Description

The map_batches release benchmark had `RAYTEST_FAIL_ON_WORKER_OOM=0`.
After we landed some changes to minimize memory bloat like
ray-project#62302, the test no longer OOMs,
so I'm re-enabling the flag.

## Related issues

None

Signed-off-by: Balaji Veeramani <bveeramani@berkeley.edu>
Signed-off-by: phattruong <23120318@student.hcmus.edu.vn>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

go add ONLY when ready to merge, run all tests

2 participants