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

DATA RACE in the TestInterruptedDuringSpilling #50688

Closed
Tracked by #41316
hawkingrei opened this issue Jan 24, 2024 · 1 comment · Fixed by #50693
Closed
Tracked by #41316

DATA RACE in the TestInterruptedDuringSpilling #50688

hawkingrei opened this issue Jan 24, 2024 · 1 comment · Fixed by #50693
Labels
component/test severity/minor sig/execution SIG execution type/bug The issue is confirmed as a bug.

Comments

@hawkingrei
Copy link
Member

hawkingrei commented Jan 24, 2024

Bug Report

Please answer these questions before submitting your issue. Thanks!

1. Minimal reproduce step (Required)

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

3. What did you see instead (Required)

==================
WARNING: DATA RACE
Read at 0x00c000616018 by goroutine 36:
  github.com/pingcap/tidb/pkg/util/chunk.TestInterruptedDuringSpilling()
      pkg/util/chunk/row_container_test.go:543 +0x5db
  github.com/pingcap/tidb/pkg/util/chunk.TestInterruptedDuringSpilling()
      pkg/util/chunk/row_container_test.go:542 +0x5c9
  testing.tRunner()
      GOROOT/src/testing/testing.go:1595 +0x261
  testing.(*T).Run.func1()
      GOROOT/src/testing/testing.go:1648 +0x44
Previous write at 0x00c000616018 by goroutine 37:
  github.com/pingcap/tidb/pkg/util/chunk.TestInterruptedDuringSpilling.func1()
      pkg/util/chunk/row_container_test.go:540 +0x87
Goroutine 36 (running) created at:
  testing.(*T).Run()
      GOROOT/src/testing/testing.go:1648 +0x845
  testing.runTests.func1()
      GOROOT/src/testing/testing.go:2054 +0x84
  testing.tRunner()
      GOROOT/src/testing/testing.go:1595 +0x261
  testing.runTests()
      GOROOT/src/testing/testing.go:2052 +0x8ad
  testing.(*M).Run()
      GOROOT/src/testing/testing.go:1925 +0xcd7
  github.com/pingcap/tidb/pkg/util/chunk.wrapper.Run()
      pkg/util/chunk/main_test.go:55 +0x64
  github.com/pingcap/tidb/pkg/util/chunk.(*wrapper).Run()
      <autogenerated>:1 +0x4a
  go.uber.org/goleak.VerifyTestMain()
      external/org_uber_go_goleak/testmain.go:53 +0x64
  github.com/pingcap/tidb/pkg/util/chunk.TestMain()
      pkg/util/chunk/main_test.go:41 +0x556
  main.main()
      bazel-out/k8-fastbuild/bin/pkg/util/chunk/chunk_test_/testmain.go:417 +0x5d1
Goroutine 37 (finished) created at:
  github.com/pingcap/tidb/pkg/util/chunk.TestInterruptedDuringSpilling()
      pkg/util/chunk/row_container_test.go:537 +0x5b8
  testing.tRunner()
      GOROOT/src/testing/testing.go:1595 +0x261
  testing.(*T).Run.func1()
      GOROOT/src/testing/testing.go:1648 +0x44
================== 

4. What is your TiDB version? (Required)

@hawkingrei hawkingrei added type/bug The issue is confirmed as a bug. component/test labels Jan 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/test severity/minor 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