tidb-x: add statement summary system variables (#23001) - #23897
ti-chi-bot wants to merge 16 commits into
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 4 included reviews per hour; 3 remain after this review. 📝 WalkthroughWalkthroughThe documentation adds TiDB Cloud behavior details for two GLOBAL Boolean statement-summary variables, including support restrictions, defaults, aggregation behavior, persistence behavior, and history-query effects. ChangesStatement Summary Variables
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~5 minutes Change: Other Suggested reviewers: Merge Risk: ⚪ Minimal · up to This change adds documentation for statement-summary variables and is ready to merge. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Description checkExplanation The description explains the documentation changes, lists the affected variables and files, identifies the source PR, and records validation checks. It does not include the required TiDB version selection section, although the change targets the release-8.5 branch.
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Advanced
Run ID: 73e03aeb-e5de-4e00-9a6e-1cf91692b5dc
📒 Files selected for processing (3)
statement-summary-tables.mdsystem-variable-reference.mdsystem-variables.md
Included review availability: Your plan provides up to 4 included reviews per hour; 3 remain after this review.
24cfb7c to
ddd649d
Compare
This is an automated cherry-pick of #23001
What is changed
This PR is translated from: pingcap/docs-cn#21694
Add English documentation for two statement summary system variables, corresponding to pingcap/docs-cn#21694:
Updated files:
Check List
Tests:
Summary by CodeRabbit