[ray_client]: Set gRPC max message size to 4GiB #14063

barakmich · 2021-02-11T22:31:49Z

Why are these changes needed?

Closes #13912 in a simple manner

I've run scripts/format.sh to lint the changes in this PR.
I've included any doc changes needed for https://docs.ray.io/en/master/.
I've made sure the tests are passing. Note that there might be a few flaky tests, see the recent failures at https://flakey-tests.ray.io/
Testing Strategy
- Unit tests
- Release tests
- This PR is not tested :(

Change-Id: Id4d6887cdd90dd761dd25248f10f104701462667

ericl

LGTM assuming tests pass

ericl · 2021-02-12T01:56:19Z

???
E OverflowError: value too large to convert to int

Heh, maybe 2GB?

Change-Id: I71625ed3cffd9d8b3d7d3d7a981bb4dda00ed0a1

[ray_client]: Set gRPC max message size to 4GiB

48e56a3

Change-Id: Id4d6887cdd90dd761dd25248f10f104701462667

barakmich requested a review from ericl February 11, 2021 22:31

barakmich assigned ericl Feb 11, 2021

ericl approved these changes Feb 11, 2021

View reviewed changes

ericl added the @author-action-required The PR author is responsible for the next step. Remove tag to send back to the reviewer. label Feb 12, 2021

barakmich and others added 4 commits February 15, 2021 21:33

reduce size

e559a0a

Change-Id: I71625ed3cffd9d8b3d7d3d7a981bb4dda00ed0a1

Update test_basic_2.py

a6fa9c6

Update test_advanced.py

93a92fc

Update test_basic.py

610c60d

ericl removed the @author-action-required The PR author is responsible for the next step. Remove tag to send back to the reviewer. label Feb 16, 2021

ericl merged commit edf2458 into ray-project:master Feb 16, 2021