[core][serialization] Fix registering serializer before initializing Ray. #23031

suquark · 2022-03-10T21:42:31Z

Why are these changes needed?

Fix the issue mentioned in #22984

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 :(

suquark · 2022-03-11T18:24:16Z

@rkooo567 strangely, I find tests with shutdown_only still have ray initialized. do you know why? (or is this expected behavior?)

suquark · 2022-03-11T23:13:05Z

The CI error seems not related. I'll merge the PR.

suquark added 2 commits March 10, 2022 13:24

Support registering serializer before initializing Ray.

48f09cc

add test

ebb1337

suquark assigned rkooo567 Mar 10, 2022

fix

6ff8c9d

rkooo567 approved these changes Mar 11, 2022

View reviewed changes

suquark added 2 commits March 11, 2022 00:09

fix

dab9e86

fix

52c6121

suquark merged commit be7ccb7 into ray-project:master Mar 11, 2022

suquark deleted the fix_registering_serializer branch March 11, 2022 23:13