Skip to content

[CoreWorker][deprecate run_function_on_all_workers 2/n] announcing deprecation of run_function_on_all_workers#31528

Merged
scv119 merged 2 commits intoray-project:masterfrom
scv119:run-on-all-workers
Jan 9, 2023
Merged

[CoreWorker][deprecate run_function_on_all_workers 2/n] announcing deprecation of run_function_on_all_workers#31528
scv119 merged 2 commits intoray-project:masterfrom
scv119:run-on-all-workers

Conversation

@scv119
Copy link
Copy Markdown
Contributor

@scv119 scv119 commented Jan 9, 2023

Why are these changes needed?

Warn user that run_function_on_all_workers will be deleted by Ray 2.4. Ray core no longer uses this function after #31383

Related issue number

Checks

@scv119 scv119 marked this pull request as ready for review January 9, 2023 05:45
@scv119 scv119 changed the title [CoreWorker] announcing deprecation of run_function_on_all_workers [CoreWorker][deprecate run_function_on_all_workers 2/n] announcing deprecation of run_function_on_all_workers Jan 9, 2023
Copy link
Copy Markdown
Member

@cadedaniel cadedaniel left a comment

Choose a reason for hiding this comment

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

Do you know what This API will be deleted once we move the working dir init away. is referring to?

Comment thread python/ray/_private/worker.py Outdated
Co-authored-by: Cade Daniel <[email protected]>
Signed-off-by: Chen Shen <[email protected]>
@scv119
Copy link
Copy Markdown
Contributor Author

scv119 commented Jan 9, 2023

Do you know what This API will be deleted once we move the working dir init away. is referring to?

yup, this is exactly what #31383 does

@rkooo567 rkooo567 added the @author-action-required The PR author is responsible for the next step. Remove tag to send back to the reviewer. label Jan 9, 2023
@scv119 scv119 merged commit c4f6a68 into ray-project:master Jan 9, 2023
AmeerHajAli pushed a commit that referenced this pull request Jan 12, 2023
…precation of run_function_on_all_workers (#31528)

Warn user that run_function_on_all_workers will be deleted by Ray 2.4. Ray core no longer uses this function after #31383
Signed-off-by: Chen Shen <[email protected]>
Co-authored-by: Cade Daniel <[email protected]>
scv119 added a commit that referenced this pull request Jun 8, 2023
…_on_all_workers (#30895)

This function is deprecated. The only use of run_funcion_on_all_workers in ray core has been replaced by #31383
Delete it to make our worker prestart change simpler.
This PR depends on #31383 #31528
arvind-chandra pushed a commit to lmco/ray that referenced this pull request Aug 31, 2023
…_on_all_workers (ray-project#30895)

This function is deprecated. The only use of run_funcion_on_all_workers in ray core has been replaced by ray-project#31383
Delete it to make our worker prestart change simpler.
This PR depends on ray-project#31383 ray-project#31528

Signed-off-by: e428265 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

@author-action-required The PR author is responsible for the next step. Remove tag to send back to the reviewer.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants