[runtime env] remove the API ray.runtime_env.get_current_runtime_env#22620
[runtime env] remove the API ray.runtime_env.get_current_runtime_env#22620SongGuyang wants to merge 2 commits intoray-project:masterfrom
Conversation
|
@edoakes There is a new issue I want to discuss: This use case is a little weird because |
|
@SongGuyang it's not safe to return a reference to the worker's runtime_env anyways (don't want users messing with it), so I think let's just document that it returns a copy |
|
linter failing on docs |
|
@SongGuyang ping to fix tests and get this merged |
|
@edoakes do we still need this PR? The previous PR has been reverted. |
|
Yes we should still merge this; it updates tests and removes the currently unused code |
|
Oh, sorry, now I realize that this makes no sense after the revert.... I was just looking at the diff but didn't realize it doesn't apply to master. My bad. |
|
OK. |
Why are these changes needed?
Remove the API
ray.runtime_env.get_current_runtime_env()and useray.get_runtime_context().runtime_envinstead.Related issue number
#22594
Checks
scripts/format.shto lint the changes in this PR.