-
Notifications
You must be signed in to change notification settings - Fork 65
support wait tiflash replica ready #142
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
support wait tiflash replica ready #142
Conversation
Signed-off-by: Weizhen Wang <wangweizhen@pingcap.com>
6b1f482 to
fe11a09
Compare
Signed-off-by: Weizhen Wang <wangweizhen@pingcap.com>
Signed-off-by: Weizhen Wang <wangweizhen@pingcap.com>
YangKeao
left a comment
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
src/type.go
Outdated
| "single_query": Q_SINGLE_QUERY, | ||
| "begin_concurrent": Q_BEGIN_CONCURRENT, | ||
| "end_concurrent": Q_END_CONCURRENT, | ||
| "wait_tiflash_replica_ready": Q_WAIT_TIFLASH_REPLICA_READY, |
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.
make fmt?
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.
Update
| if q.tp == Q_UNKNOWN || q.tp == Q_COMMENT_WITH_COMMAND { | ||
| switch q.tp { | ||
| case Q_COMMENT_WITH_COMMAND: | ||
| if err := q.getQueryType(s); err != nil { |
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.
shoud use s or realS?
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.
If I use realS, it will be unavailable for this functions.
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.
Q_COMMENT_WITH_COMMAND will remove the -- prefix and save command name into s.
bb7133
left a comment
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
test in https://github.com/PingCAP-QE/planreplayertest/pull/23
Because adding TiFlash to TiDB is an asynchronous process, to ensure the stability of the test, we need to wait for all replicas to be ready.