Skip to content

ddl: fix secondary index len check for clustered index#23710

Merged
ti-chi-bot merged 14 commits intopingcap:masterfrom
lysu:dev-add-index-check
Apr 1, 2021
Merged

ddl: fix secondary index len check for clustered index#23710
ti-chi-bot merged 14 commits intopingcap:masterfrom
lysu:dev-add-index-check

Conversation

@lysu
Copy link
Contributor

@lysu lysu commented Mar 30, 2021 •
edited
Loading

What problem does this PR solve?

Issue Number: close #23631

Problem Summary:

What is changed and how it works?

What's Changed, How it Works:

calculate non-unique secondary index len = pk len + index len

when

  • create table
  • create index
  • change column len for index
  • change column len for pk

Related changes

  • Need to cherry-pick to the release branch 5.0
  • Need update document

Check List

Tests

  • Unit test

Side effects

  • n/a

Release note

  • No release note

This change is Reviewable

@lysu lysu added the type/bugfix This PR fixes a bug. label Mar 30, 2021
@ti-chi-bot ti-chi-bot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Mar 30, 2021
@lysu
Copy link
Contributor Author

lysu commented Mar 30, 2021

/run-all-tests

@lysu lysu requested review from tangenta and wjhuang2016 March 30, 2021 14:30
@lysu
Copy link
Contributor Author

lysu commented Mar 30, 2021

/rebuild

@lysu lysu marked this pull request as ready for review March 30, 2021 14:50
@ti-chi-bot ti-chi-bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Mar 30, 2021
@github-actions github-actions bot added the sig/sql-infra SIG: SQL Infra label Mar 30, 2021
@zhouqiang-cl
Copy link
Contributor

zhouqiang-cl commented Mar 30, 2021

/run_check_dev_2

1 similar comment
@lysu
Copy link
Contributor Author

lysu commented Mar 30, 2021

/run_check_dev_2

@lysu lysu added this to the v5.0.0 ga milestone Mar 30, 2021
@lysu
Copy link
Contributor Author

lysu commented Mar 30, 2021

/run-all-tests

@lysu lysu requested a review from zimulala March 31, 2021 01:50
@lysu
Copy link
Contributor Author

lysu commented Mar 31, 2021

@zimulala @tangenta @wjhuang2016 PTAL if free, we need cherry-pick this to 5.0 today

zimulala
zimulala reviewed Mar 31, 2021
View reviewed changes
@lysu lysu requested a review from zimulala March 31, 2021 04:55
@tangenta
Copy link
Contributor

tangenta commented Mar 31, 2021

/lgtm

@ti-chi-bot ti-chi-bot added the status/LGT1 Indicates that a PR has LGTM 1. label Mar 31, 2021
zimulala
zimulala reviewed Mar 31, 2021
View reviewed changes
@lysu
Copy link
Contributor Author

lysu commented Mar 31, 2021

/run-all-tests

1 similar comment
@lysu
Copy link
Contributor Author

lysu commented Mar 31, 2021

/run-all-tests

@lysu lysu requested a review from zimulala March 31, 2021 14:24
@lysu
Copy link
Contributor Author

lysu commented Mar 31, 2021

/run-unit-test

1 similar comment
@lysu
Copy link
Contributor Author

lysu commented Mar 31, 2021

/run-unit-test

wjhuang2016
wjhuang2016 reviewed Mar 31, 2021
View reviewed changes
wjhuang2016
wjhuang2016 reviewed Mar 31, 2021
View reviewed changes
@lysu lysu requested a review from wjhuang2016 March 31, 2021 15:36
@ti-chi-bot ti-chi-bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Apr 1, 2021
@lysu lysu requested a review from tangenta April 1, 2021 11:02
@lysu
Copy link
Contributor Author

lysu commented Apr 1, 2021

/run-all-tests

@lysu
Copy link
Contributor Author

lysu commented Apr 1, 2021

/merge

@ti-chi-bot
Copy link
Member

ti-chi-bot commented Apr 1, 2021

This pull request has been accepted and is ready to merge.

DetailsCommit hash: fd3fb3e

@ti-chi-bot ti-chi-bot added the status/can-merge Indicates a PR has been approved by a committer. label Apr 1, 2021
@lysu
Copy link
Contributor Author

lysu commented Apr 1, 2021

/run-tics-test

@lysu
Copy link
Contributor Author

lysu commented Apr 1, 2021

/run-all-tests

@lysu
Copy link
Contributor Author

lysu commented Apr 1, 2021

/run-all-tests

@ti-chi-bot ti-chi-bot merged commit 91bcb21 into pingcap:master Apr 1, 2021
@lysu lysu deleted the dev-add-index-check branch April 1, 2021 13:03
ti-srebot pushed a commit to ti-srebot/tidb that referenced this pull request Apr 1, 2021
@lysu @ti-srebot
cherry pick pingcap#23710 to release-5.0
1c6e7d8 
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@ti-srebot ti-srebot mentioned this pull request Apr 1, 2021
Merged
@ti-srebot
Copy link
Contributor

ti-srebot commented Apr 1, 2021

cherry pick to release-5.0 in PR #23811

ti-chi-bot pushed a commit that referenced this pull request Apr 1, 2021
@ti-srebot
ddl: fix secondary index len check for clustered index (#23710) (#23811)
22ea38b
@TonsnakeLin TonsnakeLin mentioned this pull request Nov 10, 2021
Merged
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@zimulala zimulala Awaiting requested review from zimulala

@wjhuang2016 wjhuang2016 Awaiting requested review from wjhuang2016

@tangenta tangenta Awaiting requested review from tangenta

Assignees

No one assigned

Labels

needs-cherry-pick-release-5.0 priority/release-blocker This issue blocks a release. Please solve it ASAP. sig/sql-infra SIG: SQL Infra size/L Denotes a PR that changes 100-499 lines, ignoring generated files. status/can-merge Indicates a PR has been approved by a committer. status/LGT2 Indicates that a PR has LGTM 2. type/bugfix This PR fixes a bug.

Projects

None yet

Milestone

v5.0.0 ga

Development

Successfully merging this pull request may close these issues.

Non-unique index key may exceed default key size limit of tikv when using clustered index

7 participants

@lysu @zhouqiang-cl @tangenta @wjhuang2016 @ti-chi-bot @ti-srebot @zimulala