Skip to content

[core] move test_task_events*.py to large size#52259

Merged
jjyao merged 1 commit intomasterfrom
lonnie-250411-bigevents
Apr 11, 2025
Merged

[core] move test_task_events*.py to large size#52259
jjyao merged 1 commit intomasterfrom
lonnie-250411-bigevents

Conversation

@aslonnie
Copy link
Copy Markdown
Collaborator

they cannot reliably finish in 300 seconds anymore, especially when running with python 3.12

they cannot reliably finish in 300 seconds anymore, especially
when running with python 3.12

Signed-off-by: Lonnie Liu <lonnie@anyscale.com>
@aslonnie aslonnie requested review from dayshah, edoakes and jjyao April 11, 2025 18:05
@dayshah
Copy link
Copy Markdown
Contributor

dayshah commented Apr 11, 2025

python 3.12 and redis ^

@aslonnie aslonnie added the go add ONLY when ready to merge, run all tests label Apr 11, 2025
@aslonnie
Copy link
Copy Markdown
Collaborator Author

should we merge this?

@jjyao jjyao merged commit 7401b4d into master Apr 11, 2025
5 checks passed
@jjyao jjyao deleted the lonnie-250411-bigevents branch April 11, 2025 20:15
@jjyao
Copy link
Copy Markdown
Contributor

jjyao commented Apr 11, 2025

cc @MengjinYan to see if there is any potential perf regression.

han-steve pushed a commit to han-steve/ray that referenced this pull request Apr 11, 2025
Signed-off-by: Lonnie Liu <lonnie@anyscale.com>
Signed-off-by: Steve Han <stevehan2001@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-backlog go add ONLY when ready to merge, run all tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants