-
Notifications
You must be signed in to change notification settings - Fork 355
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
[CELEBORN-846][FOLLOWUP] Fix broken link caused by unknown RPC. #1794
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1794 +/- ##
==========================================
- Coverage 46.60% 46.49% -0.11%
==========================================
Files 162 162
Lines 10079 10103 +24
Branches 928 929 +1
==========================================
Hits 4696 4696
- Misses 5073 5097 +24
Partials 310 310
... and 1 file with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
736a751
to
bf69e85
Compare
bf69e85
to
753629c
Compare
common/src/main/scala/org/apache/celeborn/common/protocol/message/ControlMessages.scala
Outdated
Show resolved
Hide resolved
…age/ControlMessages.scala Co-authored-by: Cheng Pan <[email protected]>
Co-authored-by: Cheng Pan <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks! Merging to main/0.3
### What changes were proposed in this pull request? Keep ReleaseSlots RPC to make sure that 0.3 client can worker with 0.3.1-SNAPSHOT and 0.4.0-SNAPSHOT. This PR will need to merged into main and branch-0.3. ### Why are the changes needed? Ditto. ### Does this PR introduce _any_ user-facing change? NO. ### How was this patch tested? GA and cluster. Closes #1794 from FMX/CELEBORN-846-FOLLOWUP. Lead-authored-by: mingji <[email protected]> Co-authored-by: Ethan Feng <[email protected]> Signed-off-by: zky.zhoukeyong <[email protected]> (cherry picked from commit 7d0e257) Signed-off-by: zky.zhoukeyong <[email protected]>
What changes were proposed in this pull request?
Keep ReleaseSlots RPC to make sure that 0.3 client can worker with 0.3.1-SNAPSHOT and 0.4.0-SNAPSHOT.
This PR will need to merged into main and branch-0.3.
Why are the changes needed?
Ditto.
Does this PR introduce any user-facing change?
NO.
How was this patch tested?
GA and cluster.