Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

decodeIndexKvForClusteredIndexVersion1 function cause strings: negative Repeat count #47115

Closed
dua01 opened this issue Sep 20, 2023 · 6 comments · Fixed by #53034
Closed

decodeIndexKvForClusteredIndexVersion1 function cause strings: negative Repeat count #47115

dua01 opened this issue Sep 20, 2023 · 6 comments · Fixed by #53034

Comments

@dua01
Copy link

dua01 commented Sep 20, 2023

Bug Report

1. Minimal reproduce step (Required)

create table t (a int, b int, k varchar(64), primary key (a, b), key k (k));
insert value which k is end of over 127 spaces.
OR
table/tables/index_test.go:66 Add 128 spaces behind abc and run test

2. What did you expect to see? (Required)

Test Pass

3. What did you see instead (Required)

panic: strings: negative Repeat count

4. What is your TiDB version? (Required)

Tidb 5.0+

@dua01 dua01 added the type/bug The issue is confirmed as a bug. label Sep 20, 2023
@jebter
Copy link

jebter commented Sep 21, 2023

/sig executor
/severity major

@ti-chi-bot
Copy link

ti-chi-bot bot commented Sep 21, 2023

@jebter: The label(s) sig/executor cannot be applied, because the repository doesn't have them.

In response to this:

/sig executor
/severity major

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the ti-community-infra/tichi repository.

@ti-chi-bot ti-chi-bot bot added severity/major may-affects-5.3 This bug maybe affects 5.3.x versions. may-affects-5.4 This bug maybe affects 5.4.x versions. may-affects-6.1 may-affects-6.5 may-affects-7.1 labels Sep 21, 2023
@jebter
Copy link

jebter commented Sep 21, 2023

/sig executor

@ti-chi-bot
Copy link

ti-chi-bot bot commented Sep 21, 2023

@jebter: The label(s) sig/executor cannot be applied, because the repository doesn't have them.

In response to this:

/sig executor

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the ti-community-infra/tichi repository.

@jebter
Copy link

jebter commented Sep 21, 2023

/sig execution

@ti-chi-bot ti-chi-bot bot added the sig/execution SIG execution label Sep 21, 2023
@zanmato1984 zanmato1984 added sig/transaction SIG:Transaction and removed sig/execution SIG execution labels Oct 9, 2023
@zanmato1984
Copy link
Contributor

Transfer to sig/transaction.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants