Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[core] Return failure for failed runtime env dereference #50003

Merged
merged 5 commits into from
Jan 24, 2025

Conversation

dentiny
Copy link
Contributor

@dentiny dentiny commented Jan 22, 2025

Dereference is not supposed to fail, if only warning message is logged, it's hard to detected failed precondition.

@dentiny dentiny added the go add ONLY when ready to merge, run all tests label Jan 22, 2025
@dentiny dentiny requested review from jjyao and rynewang January 22, 2025 09:38
@dentiny dentiny force-pushed the hjiang/return-failure-for-deref branch from 8141739 to 9215e16 Compare January 22, 2025 09:39
@dentiny dentiny force-pushed the hjiang/return-failure-for-deref branch 2 times, most recently from 96b8df7 to 4cd4f7c Compare January 23, 2025 23:50
@dentiny dentiny force-pushed the hjiang/return-failure-for-deref branch from 4cd4f7c to ae4fa80 Compare January 23, 2025 23:51
@dentiny dentiny requested a review from rynewang January 23, 2025 23:51
@dentiny dentiny requested a review from rynewang January 24, 2025 21:19
@rynewang rynewang enabled auto-merge (squash) January 24, 2025 21:23
@rynewang rynewang merged commit d27f68c into ray-project:master Jan 24, 2025
6 checks passed
erictang000 pushed a commit to erictang000/ray that referenced this pull request Jan 27, 2025
…#50003)

Dereference is not supposed to fail, if only warning message is logged,
it's hard to detected failed precondition.

---------

Signed-off-by: dentiny <[email protected]>
Signed-off-by: Eric Tang <[email protected]>
srinathk10 pushed a commit that referenced this pull request Feb 2, 2025
Dereference is not supposed to fail, if only warning message is logged,
it's hard to detected failed precondition.

---------

Signed-off-by: dentiny <[email protected]>
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants