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

Send hearbeat along with status update between GCS and raylet #48837

Closed
dentiny opened this issue Nov 21, 2024 · 0 comments
Closed

Send hearbeat along with status update between GCS and raylet #48837

dentiny opened this issue Nov 21, 2024 · 0 comments
Assignees
Labels
core Issues that should be addressed in Ray Core enhancement Request for new feature and/or capability P2 Important issue, but not time-critical

Comments

@dentiny
Copy link
Contributor

dentiny commented Nov 21, 2024

Description

As titled, to reduce communication between raylet and GCS.

Use case

No response

@dentiny dentiny added enhancement Request for new feature and/or capability P2 Important issue, but not time-critical triage Needs triage (eg: priority, bug/not-bug, and owning component) core Issues that should be addressed in Ray Core labels Nov 21, 2024
@dentiny dentiny self-assigned this Nov 21, 2024
@jjyao jjyao removed the triage Needs triage (eg: priority, bug/not-bug, and owning component) label Nov 25, 2024
rynewang pushed a commit that referenced this issue Dec 21, 2024
…at NodeManager (#49235)

Issue reference: #48837
This PR simply combines two existing API call: client registration and
port announcement into one, with no functionality change.

The plan is to
- Add new APIs at callee side, so we don't have production effect;
- Do no-op code refactor at caller side, so it's easier to refactor and
merge API calls
- Connect caller and callee with the new API calls, after (1) and (2)
the code structure should be at a clean state which is friendly and easy
for API merge.

---------

Signed-off-by: hjiang <hjiang@anyscale.com>
@dentiny dentiny closed this as completed Dec 23, 2024
anyadontfly pushed a commit to anyadontfly/ray that referenced this issue Feb 13, 2025
…at NodeManager (ray-project#49235)

Issue reference: ray-project#48837
This PR simply combines two existing API call: client registration and
port announcement into one, with no functionality change.

The plan is to
- Add new APIs at callee side, so we don't have production effect;
- Do no-op code refactor at caller side, so it's easier to refactor and
merge API calls
- Connect caller and callee with the new API calls, after (1) and (2)
the code structure should be at a clean state which is friendly and easy
for API merge.

---------

Signed-off-by: hjiang <hjiang@anyscale.com>
Signed-off-by: Puyuan Yao <williamyao034@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Issues that should be addressed in Ray Core enhancement Request for new feature and/or capability P2 Important issue, but not time-critical
Projects
None yet
Development

No branches or pull requests

2 participants