Skip to content

support wait tiflash replica ready#142

Merged
bb7133 merged 6 commits into
pingcap:masterfrom
hawkingrei:add_wait_tiflash_replica_ready
Dec 10, 2025
Merged

support wait tiflash replica ready#142
bb7133 merged 6 commits into
pingcap:masterfrom
hawkingrei:add_wait_tiflash_replica_ready

Conversation

@hawkingrei
Copy link
Copy Markdown
Member

@hawkingrei hawkingrei commented Dec 8, 2025

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.

Signed-off-by: Weizhen Wang <wangweizhen@pingcap.com>
@hawkingrei hawkingrei force-pushed the add_wait_tiflash_replica_ready branch from 6b1f482 to fe11a09 Compare December 8, 2025 10:39
Signed-off-by: Weizhen Wang <wangweizhen@pingcap.com>
Signed-off-by: Weizhen Wang <wangweizhen@pingcap.com>
Signed-off-by: Weizhen Wang <wangweizhen@pingcap.com>
Signed-off-by: Weizhen Wang <wangweizhen@pingcap.com>
Copy link
Copy Markdown
Member

@YangKeao YangKeao left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread 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,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

make fmt?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Update

Comment thread src/query.go
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 {
Copy link
Copy Markdown
Contributor

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?

Copy link
Copy Markdown
Member Author

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.

Copy link
Copy Markdown
Member Author

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.

Signed-off-by: Weizhen Wang <wangweizhen@pingcap.com>
Copy link
Copy Markdown
Member

@bb7133 bb7133 left a comment

Choose a reason for hiding this comment

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

LGTM

@bb7133 bb7133 merged commit edd0490 into pingcap:master Dec 10, 2025
2 checks passed
dveeden pushed a commit to dveeden/mysql-tester that referenced this pull request May 22, 2026
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.

4 participants