Skip to content

Conversation

@breezewish
Copy link
Member

What problem does this PR solve?

Currently playground cluster is boot for debug-friendly by default.

What is changed and how it works?

  • Add --perf to allow the cluster boot for higher performance instead of debug friendly.
    Currently only TiFlash config is polished (and is sufficient for testing the TiDB cluster using VectorDBBench). Other config can be also added if someone need it.

  • Now there are more and more global options. It makes code hard to read. So I reorganized these options and grouped them into a single structure. In this way it is more clear and will be easy to add global options later.

Check List

Tests

  • Manual test (add detailed scripts or steps below)

Tested with --perf, --mode=tidb-cse and all works as expected.

Release notes:

playground: Add --perf for starting cluster for higher performance

Signed-off-by: Wish <breezewish@outlook.com>
@ti-chi-bot ti-chi-bot bot requested a review from kaaaaaaang May 6, 2025 07:25
@ti-chi-bot
Copy link
Contributor

ti-chi-bot bot commented May 6, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign bb7133 for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ti-chi-bot ti-chi-bot bot requested a review from srstack May 6, 2025 07:25
@ti-chi-bot ti-chi-bot bot added the size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. label May 6, 2025
@ti-chi-bot ti-chi-bot bot added the lgtm label May 6, 2025
@ti-chi-bot
Copy link
Contributor

ti-chi-bot bot commented May 6, 2025

[LGTM Timeline notifier]

Timeline:

  • 2025-05-06 08:15:35.61858071 +0000 UTC m=+342982.801483971: ☑️ agreed by xhebox.

@xhebox xhebox merged commit ba38577 into master May 6, 2025
6 of 7 checks passed
@xhebox xhebox deleted the wenxuan/perf branch May 6, 2025 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants