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

Empty groupOffset in StreamAggExec causes panic #53867

Closed
xzhangxian1008 opened this issue Jun 7, 2024 · 9 comments · Fixed by #53874
Closed

Empty groupOffset in StreamAggExec causes panic #53867

xzhangxian1008 opened this issue Jun 7, 2024 · 9 comments · Fixed by #53874
Labels
affects-5.4 This bug affects the 5.4.x(LTS) versions. affects-6.1 This bug affects the 6.1.x(LTS) versions. affects-6.5 This bug affects the 6.5.x(LTS) versions. affects-7.1 This bug affects the 7.1.x(LTS) versions. affects-7.5 This bug affects the 7.5.x(LTS) versions. affects-8.1 This bug affects the 8.1.x(LTS) versions. impact/panic severity/major sig/execution SIG execution type/bug The issue is confirmed as a bug.

Comments

@xzhangxian1008
Copy link
Contributor

xzhangxian1008 commented Jun 7, 2024

Bug Report

Please answer these questions before submitting your issue. Thanks!

1. Minimal reproduce step (Required)

create database test;
use test;
source valid.txt
source error.txt

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

no panic

3. What did you see instead (Required)

We get panic: ERROR 1105 (HY000): runtime error: index out of range [0] with length 0

4. What is your TiDB version? (Required)

a865c86

@xzhangxian1008 xzhangxian1008 added the type/bug The issue is confirmed as a bug. label Jun 7, 2024
Copy link

ti-chi-bot bot commented Jun 7, 2024

@xzhangxian1008: 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.

@xzhangxian1008
Copy link
Contributor Author

/sig execution

@ti-chi-bot ti-chi-bot bot added the sig/execution SIG execution label Jun 7, 2024
Copy link

ti-chi-bot bot commented Jun 7, 2024

@xzhangxian1008: 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.

@xzhangxian1008 xzhangxian1008 changed the title Empty groupOffset in StreamAggExec caused panic Empty groupOffset in StreamAggExec causes panic Jun 7, 2024
@xzhangxian1008
Copy link
Contributor Author

/remove-label may-affects-5.4

@ti-chi-bot ti-chi-bot bot removed the may-affects-5.4 This bug maybe affects 5.4.x versions. label Jun 18, 2024
@xzhangxian1008
Copy link
Contributor Author

/remove-label may-affects-6.1

@xzhangxian1008
Copy link
Contributor Author

/remove-label may-affects-6.5

@xzhangxian1008
Copy link
Contributor Author

/remove-label may-affects-7.1

@xzhangxian1008
Copy link
Contributor Author

/remove-label may-affects-7.5

@xzhangxian1008
Copy link
Contributor Author

/remove-label may-affects-8.1

@windtalker windtalker added affects-5.4 This bug affects the 5.4.x(LTS) versions. affects-6.1 This bug affects the 6.1.x(LTS) versions. affects-6.5 This bug affects the 6.5.x(LTS) versions. affects-7.1 This bug affects the 7.1.x(LTS) versions. affects-7.5 This bug affects the 7.5.x(LTS) versions. affects-8.1 This bug affects the 8.1.x(LTS) versions. labels Aug 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects-5.4 This bug affects the 5.4.x(LTS) versions. affects-6.1 This bug affects the 6.1.x(LTS) versions. affects-6.5 This bug affects the 6.5.x(LTS) versions. affects-7.1 This bug affects the 7.1.x(LTS) versions. affects-7.5 This bug affects the 7.5.x(LTS) versions. affects-8.1 This bug affects the 8.1.x(LTS) versions. impact/panic severity/major sig/execution SIG execution type/bug The issue is confirmed as a bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants