Skip to content

Some region migration related work#12376

Merged
OneSizeFitsQuorum merged 5 commits intoapache:masterfrom
liyuheng55555:Working/region-migration-add-check
Apr 23, 2024
Merged

Some region migration related work#12376
OneSizeFitsQuorum merged 5 commits intoapache:masterfrom
liyuheng55555:Working/region-migration-add-check

Conversation

@liyuheng55555
Copy link
Collaborator

No description provided.

OneSizeFitsQuorum

This comment was marked as outdated.

.forceUpdateConsensusGroupCache(
Collections.singletonMap(
regionId,
new ConsensusGroupHeartbeatSample(
Copy link
Contributor

Choose a reason for hiding this comment

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

should be the TRegionLeaderChangeResp. getConsensusLogicalTimestamp() for ratisconsensus?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

sure

@liyuheng55555 liyuheng55555 force-pushed the Working/region-migration-add-check branch from 0735d46 to 1356405 Compare April 23, 2024 07:18
Copy link
Contributor

@OneSizeFitsQuorum OneSizeFitsQuorum left a comment

Choose a reason for hiding this comment

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

LGTM

@OneSizeFitsQuorum OneSizeFitsQuorum merged commit b7e9d6e into apache:master Apr 23, 2024
@OneSizeFitsQuorum OneSizeFitsQuorum deleted the Working/region-migration-add-check branch April 23, 2024 10:27
HTHou pushed a commit that referenced this pull request Jun 26, 2024
SzyWilliam pushed a commit to SzyWilliam/iotdb that referenced this pull request Nov 26, 2024
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.

2 participants