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

[Improve][Manager] Optimize the HeartbeatMsg class #8392

Closed
2 tasks done
gosonzhang opened this issue Jul 3, 2023 · 2 comments · Fixed by #9512
Closed
2 tasks done

[Improve][Manager] Optimize the HeartbeatMsg class #8392

gosonzhang opened this issue Jul 3, 2023 · 2 comments · Fixed by #9512

Comments

@gosonzhang
Copy link
Contributor

Description

This parent issue is: #8391

Based on the heartbeat reporting characteristics of DataProxy, it is necessary to provide a separate heartbeat method and heartbeat information class. This class needs to support DataProxy to carry the {ip, port, protocolType} information pair of the source to the Manager as List information.

InLong Component

InLong Manager

Are you willing to submit PR?

  • Yes, I am willing to submit a PR!

Code of Conduct

@github-actions
Copy link

github-actions bot commented Sep 2, 2023

This issue is stale because it has been open for 60 days with no activity.

@github-actions github-actions bot added the stage/stale Issues or PRs that had no activity for a long time label Sep 2, 2023
@dockerzhang dockerzhang modified the milestones: 1.9.0, 1.10.0 Sep 14, 2023
@github-actions github-actions bot removed the stage/stale Issues or PRs that had no activity for a long time label Sep 15, 2023
Copy link

This issue is stale because it has been open for 60 days with no activity.

@github-actions github-actions bot added the stage/stale Issues or PRs that had no activity for a long time label Nov 14, 2023
@dockerzhang dockerzhang modified the milestones: 1.10.0, 1.11.0 Dec 5, 2023
@github-actions github-actions bot removed the stage/stale Issues or PRs that had no activity for a long time label Dec 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants