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

branch-3.0: [fix](heartbeat) fix heartbeat editlog no persist hbTime #42986

Merged
merged 1 commit into from
Nov 7, 2024

Conversation

github-actions[bot]
Copy link
Contributor

PR Body: Backend persist lastUpdateMs, it will be modified by heartbeat editlog. But heartbeat editlog not persist hbTime, and hbTime always equal 0, it will make backend's lastUpdateMs = 0 in bdb image.

fix details:

  1. heartbeat response persist hbTime;
  2. only be state change will write an editlog. but we make a change: even a backend is healthy, still write a healthy response editlog every 5 min. Inorder to make backend's lastUpdateMs periodly updated in bdb image. But notice that this change wouldn't increase real editlog num. Because heartbeat mgr will patch all fe/be's heartbeat into one editlog. Even no fe/be state change, it still write an editlog which not contains any node's response.
  3. for a dead heartbeat response, set hbTime to last succ hbTime, then replayer can set correct lastUpdateMs;
    Cherry-picked from [fix](heartbeat) fix heartbeat editlog no persist hbTime #42653

Backend persist lastUpdateMs, it will be modified by heartbeat editlog.
But heartbeat editlog not persist hbTime, and hbTime always equal 0, it
will make backend's lastUpdateMs = 0 in bdb image.

fix details:
1. heartbeat response persist hbTime;
2. only be state change will write an editlog. but we make a change:
even a backend is healthy, still write a healthy response editlog every
5 min. Inorder to make backend's lastUpdateMs periodly updated in bdb
image. But notice that this change wouldn't increase real editlog num.
Because heartbeat mgr will patch all fe/be's heartbeat into one editlog.
Even no fe/be state change, it still write an editlog which not contains
any node's response.
3. for a dead heartbeat response, set hbTime to last succ hbTime, then
replayer can set correct lastUpdateMs;
Copy link
Contributor Author

run buildall

@doris-robot
Copy link

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR

Since 2024-03-18, the Document has been moved to doris-website.
See Doris Document.

@yujun777 yujun777 closed this Nov 7, 2024
@yujun777 yujun777 reopened this Nov 7, 2024
@dataroaring
Copy link
Contributor

run buildall

@dataroaring dataroaring merged commit 7abf3c5 into branch-3.0 Nov 7, 2024
22 of 23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants