-
Notifications
You must be signed in to change notification settings - Fork 489
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
fix key work node_id && node_state #263
Conversation
PTAL @zimulala |
@@ -267,7 +265,7 @@ import ( | |||
zerofill "ZEROFILL" | |||
natural "NATURAL" | |||
|
|||
/* The following tokens belong to UnReservedKeyword. */ | |||
/* The following tokens belong to UnReservedKeyword. Notice: make sure these tokens are contained in UnReservedKeyword. */ |
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.
🤔 So turns out this is already checked in consistent_test.go
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.
🤔but it don't take effect when 'node_id' not in keyword list, I don't know why
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.
Only put node_id
in TiDBKeyword part (or other special parts) can take effect.
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.
LGTM
LGTM |
What problem does this PR solve?
fix issue pingcap/tidb#9910
treat
node_id
andnode_stats
as key workWhat is changed and how it works?
move
pump
drainer
node_id
node_stats
toNotKeywordToken
Check List
Tests
pump
drainer
node_id
node_stats
as column name)Related changes