Skip to content
This repository has been archived by the owner on Dec 1, 2022. It is now read-only.

Cache partition term #492

Merged
merged 5 commits into from
Jul 2, 2021
Merged

Cache partition term #492

merged 5 commits into from
Jul 2, 2021

Conversation

liuyu85cn
Copy link
Contributor

@liuyu85cn liuyu85cn commented Apr 8, 2021

Only save term of partition at meta is useless.

There has to be a mechanism to let all storages know each other's partition term.

This is the following PR after

Send leader term in heart beat
and
send leader term with heart and reorganize leader table

@liuyu85cn liuyu85cn marked this pull request as ready for review April 15, 2021 09:29
@liuyu85cn liuyu85cn added ready-for-review ready-for-testing PR: ready for the CI test labels Apr 15, 2021
@liuyu85cn liuyu85cn requested a review from a team April 15, 2021 09:35
panda-sheep
panda-sheep previously approved these changes Apr 16, 2021
Copy link
Contributor

@panda-sheep panda-sheep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good job

Copy link
Contributor

@bright-starry-sky bright-starry-sky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@CLAassistant
Copy link

CLAassistant commented Jun 17, 2021

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor

@critical27 critical27 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@critical27 critical27 merged commit 6b31073 into vesoft-inc:master Jul 2, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ready-for-testing PR: ready for the CI test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants