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.

Sign up for GitHub

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

Jump to bottom

[INLONG-5555][Manager] Fix missing heartbeat fields and the file task split error #5556

Merged
merged 9 commits into from
Aug 18, 2022
Merged

[INLONG-5555][Manager] Fix missing heartbeat fields and the file task split error #5556

merged 9 commits into from
Aug 18, 2022

Conversation

woofyzhao
Copy link
Contributor

@woofyzhao woofyzhao commented Aug 15, 2022 •
edited
Loading

Change list

  1. add cluster name and ip fields for agent and dataproxy heartbeat report.
  2. remove some redundant fields in file source model.
  3. fix the file task split by cluster implementation bug.
  4. remove the deletion logic from client

@woofyzhao woofyzhao changed the title [INLONG-5555][Manager] Fix missing heartbeat fields and the file task split [INLONG-5555][Manager] Fix missing heartbeat fields and the file task split error Aug 18, 2022
@healchow healchow merged commit 0222b5b into apache:master Aug 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dockerzhang dockerzhang dockerzhang left review comments

@healchow healchow healchow approved these changes

@aloyszhang aloyszhang aloyszhang approved these changes

@chestnut-c chestnut-c chestnut-c approved these changes

Assignees
No one assigned
Labels
component/agent component/dataproxy component/manager service/docker
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Bug][Manager] Fix missing heartbeat fields and stream source subtask creation
5 participants
@woofyzhao @dockerzhang @healchow @aloyszhang @chestnut-c