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

Revert "[Client] chunked get requests" #22455

Merged
merged 1 commit into from
Feb 17, 2022
Merged

Conversation

fishbone
Copy link
Contributor

Reverts #22100

linux://python/ray/tests:test_runtime_env_working_dir_remote_uri becomes very flaky after this PR.

@fishbone fishbone merged commit 83257a4 into master Feb 17, 2022
@fishbone fishbone deleted the revert-22100-chunkedgets branch February 17, 2022 00:43
@AmeerHajAli
Copy link
Contributor

@ckw017 , did we revert the reverted PR?

simonsays1980 pushed a commit to simonsays1980/ray that referenced this pull request Feb 27, 2022
Reverts ray-project#22100

linux://python/ray/tests:test_runtime_env_working_dir_remote_uri becomes very flaky after this PR.
@ckw017
Copy link
Member

ckw017 commented Feb 28, 2022

@AmeerHajAli not yet, still trying to reproduce and understand the cause of flakiness

ckw017 added a commit to ckw017/ray that referenced this pull request Mar 1, 2022
suquark pushed a commit that referenced this pull request Mar 17, 2022
* revert revertchunkedgets

* exit early if all chunks received, tighter exception handler for stream in proxy
scv119 pushed a commit that referenced this pull request Mar 30, 2022
* revert revertchunkedgets

* exit early if all chunks received, tighter exception handler for stream in proxy
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants