-
Notifications
You must be signed in to change notification settings - Fork 719
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
ReAdd peer meta. #101
ReAdd peer meta. #101
Conversation
for _, peerStoreID := range region.StoreIds { | ||
if peerStoreID == storeID { | ||
for _, peer := range region.Peers { | ||
if peer.GetStoreId() == storeID { | ||
// we can't add peer in the same store. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tip: maybe we'll allow this later, but allocate a new peer id and recycle the old peer.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why should support this?
A store can only have one peer for the region. Pd must remove first, then add again.
LGTM |
1 similar comment
LGTM |
* Fix tso server close stuck issue (tikv#6529) ref tikv#5895, close tikv#6304 Rewrite TSO gPRC/HTTP server Close(). Signed-off-by: Bin Shi <[email protected]> * mcs, tso: change keyspace group primary path. (tikv#6526) ref tikv#5895 mcs, tso: change keyspace group primary path. The path for non-default keyspace group primary election changes from "/ms/{cluster_id}/tso/{group}/primary" to "/ms/{cluster_id}/tso/keyspace_groups/election/{group}/primary". Default keyspace group keeps /ms/{cluster_id}/tso/00000/primary. Signed-off-by: Bin Shi <[email protected]> --------- Signed-off-by: Bin Shi <[email protected]>
No description provided.