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

infoschema v1 and v2 are inconsistent during TestDisaggregatedTiFlashGeneratedColumn #55462

Closed
joechenrh opened this issue Aug 16, 2024 · 6 comments · Fixed by #55463
Closed
Labels
impact/panic severity/major type/bug The issue is confirmed as a bug.

Comments

@joechenrh
Copy link
Contributor

Bug Report

Please answer these questions before submitting your issue. Thanks!

1. Minimal reproduce step (Required)

This issue is similar to #55442. Run TestDisaggregatedTiFlashGeneratedColumn using #55030.

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

Test passed

3. What did you see instead (Required)

--- FAIL: TestDisaggregatedTiFlashGeneratedColumn (0.51s)
panic: inconsistent infoschema [recovered]
        panic: inconsistent infoschema [recovered]
        panic: inconsistent infoschema

goroutine 23 [running]:
testing.tRunner.func1.2({0x1070259a0, 0x10795bfe0})
        /Users/joechenrh/deps/go/src/testing/testing.go:1631 +0x1c4
testing.tRunner.func1()
        /Users/joechenrh/deps/go/src/testing/testing.go:1634 +0x33c
panic({0x1070259a0?, 0x10795bfe0?})
        /Users/joechenrh/deps/go/src/runtime/panic.go:770 +0x124
github.com/pingcap/tidb/pkg/executor.(*ExecStmt).Exec.func1()
        /Users/joechenrh/code/tidb-test/pkg/executor/adapter.go:484 +0x428
panic({0x1070259a0?, 0x10795bfe0?})
        /Users/joechenrh/deps/go/src/runtime/panic.go:770 +0x124
github.com/pingcap/tidb/pkg/infoschema.(*InfoschemaV3).TableByID(0x14006a72360, 0x6c)
        /Users/joechenrh/code/tidb-test/pkg/infoschema/infoschema_v3.go:225 +0xb8
github.com/pingcap/tidb/pkg/infoschema.(*SessionExtendedInfoSchema).TableByID(0x14007a63b90, 0x6c)
        /Users/joechenrh/code/tidb-test/pkg/infoschema/infoschema.go:843 +0x94
github.com/pingcap/tidb/pkg/executor.buildTableReq(0x14009931880, 0x3, {0x14005c8a870, 0x1, 0x14000da5cd8?})
        /Users/joechenrh/code/tidb-test/pkg/executor/builder.go:3913 +0x160
github.com/pingcap/tidb/pkg/executor.buildNoRangeIndexLookUpReader(0x14009931880, 0x14008425340)
        /Users/joechenrh/code/tidb-test/pkg/executor/builder.go:3981 +0xc4
github.com/pingcap/tidb/pkg/executor.(*executorBuilder).buildIndexLookUpReader(0x14009931880, 0x14008425340)
        /Users/joechenrh/code/tidb-test/pkg/executor/builder.go:4043 +0xa4
github.com/pingcap/tidb/pkg/executor.(*executorBuilder).build(0x10a1a80a0?, {0x1079cb4e0?, 0x14008425340})
        /Users/joechenrh/code/tidb-test/pkg/executor/builder.go:294 +0x98c
github.com/pingcap/tidb/pkg/executor.(*executorBuilder).buildProjection(0x14009931880, 0x1400a559b20)
        /Users/joechenrh/code/tidb-test/pkg/executor/builder.go:2013 +0x68
github.com/pingcap/tidb/pkg/executor.(*executorBuilder).build(0x107a23cd8?, {0x1079cafe0?, 0x1400a559b20})
        /Users/joechenrh/code/tidb-test/pkg/executor/builder.go:276 +0x92c
github.com/pingcap/tidb/pkg/executor.(*executorBuilder).buildExplain(0x14009931880, 0x140082ac8f0)
        /Users/joechenrh/code/tidb-test/pkg/executor/builder.go:1236 +0x1ac
github.com/pingcap/tidb/pkg/executor.(*executorBuilder).build(0x107a23cd8?, {0x1079cbe60?, 0x140082ac8f0})
        /Users/joechenrh/code/tidb-test/pkg/executor/builder.go:194 +0x119c
github.com/pingcap/tidb/pkg/executor.(*ExecStmt).buildExecutor(0x14006e12870)
        /Users/joechenrh/code/tidb-test/pkg/executor/adapter.go:1210 +0x164
github.com/pingcap/tidb/pkg/executor.(*ExecStmt).Exec(0x14006e12870, {0x10799fef8, 0x14008259ce0})
        /Users/joechenrh/code/tidb-test/pkg/executor/adapter.go:555 +0x688
github.com/pingcap/tidb/pkg/session.runStmt({0x10799fe50, 0x10ad5b848}, 0x14001994008, {0x1079b4640, 0x14006e12870})
        /Users/joechenrh/code/tidb-test/pkg/session/session.go:2288 +0x344
github.com/pingcap/tidb/pkg/session.(*session).ExecuteStmt(0x14001994008, {0x10799fe50?, 0x10ad5b848?}, {0x1079b9020, 0x14009931180})
        /Users/joechenrh/code/tidb-test/pkg/session/session.go:2149 +0xd64
github.com/pingcap/tidb/pkg/testkit.(*TestKit).ExecWithContext(0x140066a0050, {0x10799fe50, 0x10ad5b848}, {0x1400693aaa0, 0x44}, {0x0, 0x0, 0x0})
        /Users/joechenrh/code/tidb-test/pkg/testkit/testkit.go:385 +0x6f4
github.com/pingcap/tidb/pkg/testkit.(*TestKit).MustQueryWithContext(0x140066a0050, {0x10799fe50, 0x10ad5b848}, {0x1400693aaa0, 0x44}, {0x0, 0x0, 0x0})
        /Users/joechenrh/code/tidb-test/pkg/testkit/testkit.go:196 +0xc8
github.com/pingcap/tidb/pkg/testkit.(*TestKit).MustQuery(0x106341132?, {0x1400693aaa0?, 0x14000da7dc0?}, {0x0?, 0x1?, 0x0?})
        /Users/joechenrh/code/tidb-test/pkg/testkit/testkit.go:174 +0x78
github.com/pingcap/tidb/pkg/executor/test/tiflashtest.TestDisaggregatedTiFlashGeneratedColumn.func1(0x0)
        /Users/joechenrh/code/tidb-test/pkg/executor/test/tiflashtest/tiflash_test.go:1601 +0x2b4
github.com/pingcap/tidb/pkg/executor/test/tiflashtest.TestDisaggregatedTiFlashGeneratedColumn(0x1400193a9c0)
        /Users/joechenrh/code/tidb-test/pkg/executor/test/tiflashtest/tiflash_test.go:1668 +0x4f0
testing.tRunner(0x1400193a9c0, 0x10794df68)
        /Users/joechenrh/deps/go/src/testing/testing.go:1689 +0xec
created by testing.(*T).Run in goroutine 1
        /Users/joechenrh/deps/go/src/testing/testing.go:1742 +0x318
FAIL    github.com/pingcap/tidb/pkg/executor/test/tiflashtest   1.642s

4. What is your TiDB version? (Required)

@joechenrh joechenrh added the type/bug The issue is confirmed as a bug. label Aug 16, 2024
@joechenrh
Copy link
Contributor Author

/label severity/major

Copy link

ti-chi-bot bot commented Aug 19, 2024

@joechenrh: The label(s) severity/major cannot be applied. These labels are supported: fuzz/sqlancer, challenge-program, compatibility-breaker, first-time-contributor, contribution, good first issue, correctness, duplicate, proposal, security, ok-to-test, needs-ok-to-test, needs-more-info, needs-cherry-pick-release-5.4, needs-cherry-pick-release-6.1, needs-cherry-pick-release-6.5, needs-cherry-pick-release-7.1, needs-cherry-pick-release-7.5, needs-cherry-pick-release-8.1, needs-cherry-pick-release-8.3, affects-5.4, affects-6.1, affects-6.5, affects-7.1, affects-7.5, affects-8.1, affects-8.3, may-affects-5.4, may-affects-6.1, may-affects-6.5, may-affects-7.1, may-affects-7.5, may-affects-8.1.

In response to this:

/label 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.

@lance6716
Copy link
Contributor

/severity major

@joechenrh
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 Aug 19, 2024
@joechenrh
Copy link
Contributor Author

/remove-label may-affects-6.1 may-affects-6.5 may-affects-7.1 may-affects-7.5 may-affects-8.1

@joechenrh
Copy link
Contributor Author

/remove-label may-affects-6.1
/remove-label may-affects-6.5
/remove-label may-affects-7.1
/remove-label may-affects-7.5
/remove-label may-affects-8.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
impact/panic severity/major type/bug The issue is confirmed as a bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants