Skip to content

system-variables: Merge with tidb-specific-sysvars (#3152)#3261

Merged
ti-srebot merged 4 commits intopingcap:release-4.0from
ti-srebot:release-4.0-7c3206dd7267
Jul 14, 2020
Merged

system-variables: Merge with tidb-specific-sysvars (#3152)#3261
ti-srebot merged 4 commits intopingcap:release-4.0from
ti-srebot:release-4.0-7c3206dd7267

Conversation

@ti-srebot
Copy link
Contributor

cherry-pick #3152 to release-4.0


What is changed, added or deleted? (Required)

This PR proposes merging the system-variables and tidb-specific-system-variables documents together.

The motivation for this, is that they both have the same semantics. It is just a namespace that differs, but this can clearly be visible by sorting the variables in alphabetical order (something else I've done.)

In the process of creating this PR, I discovered that several sysvars are undocumented. I have created #3155 to handle separately.

Also, SHARD_ROW_ID_BITS was documented as a system variable but it is not. I have moved it to the statement reference for CREATE TABLE and ALTER TABLE.

Which TiDB version(s) do your changes apply to? (Required)

  • master (the latest development version)
  • v4.0 (TiDB 4.0 versions)
  • v3.1 (TiDB 3.1 versions)
  • v3.0 (TiDB 3.0 versions)
  • v2.1 (TiDB 2.1 versions)

What is the related PR or file link(s)?

This is original work.

Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@ti-srebot ti-srebot added contribution This PR is from a community contributor. size/large Changes of a large size. type/cherry-pick-for-release-4.0 This PR is cherry-picked to release-4.0 from a source PR. labels Jul 14, 2020
@ti-srebot ti-srebot assigned ghost Jul 14, 2020
@yikeke
Copy link
Contributor

yikeke commented Jul 14, 2020

Please help delete tidb-specific-system-variables.md. @nullnotnil (P.S. Cherry-picker bot can't delete files because of a known bug)

@yikeke yikeke self-requested a review July 14, 2020 02:12
Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
@ghost
Copy link

ghost commented Jul 14, 2020

@yikeke tidb-specific-system-variables.md has been deleted, thanks for the heads up. This is ready for review.

Copy link
Contributor

@yikeke yikeke left a comment

Choose a reason for hiding this comment

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

LGTM

@ti-srebot ti-srebot added the status/LGT1 Indicates that a PR has LGTM 1. label Jul 14, 2020
@yikeke
Copy link
Contributor

yikeke commented Jul 14, 2020

/merge

@ti-srebot ti-srebot added the status/can-merge Indicates a PR has been approved by a committer. label Jul 14, 2020
@ti-srebot
Copy link
Contributor Author

/run-all-tests

@ti-srebot ti-srebot merged commit 89af0ea into pingcap:release-4.0 Jul 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contribution This PR is from a community contributor. size/large Changes of a large size. status/can-merge Indicates a PR has been approved by a committer. status/LGT1 Indicates that a PR has LGTM 1. type/cherry-pick-for-release-4.0 This PR is cherry-picked to release-4.0 from a source PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants