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 rules_go/go/tools/bzltestutil #50216

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

DATA RACE in the rules_go/go/tools/bzltestutil #50216

hawkingrei opened this issue Jan 9, 2024 · 1 comment · Fixed by #50228
Labels
component/test good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines. severity/moderate type/bug The issue is confirmed as a bug.

Comments

@hawkingrei
Copy link
Member

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 0x00c00088a360 by goroutine 53:
  github.com/bazelbuild/rules_go/go/tools/bzltestutil.Wrap.func1()
      external/io_bazel_rules_go/go/tools/bzltestutil/wrap.go:140 +0x9b
Previous write at 0x00c00088a360 by main goroutine:
  os/exec.(*Cmd).Start()
      GOROOT/src/os/exec/exec.go:693 +0xa57
  os/exec.(*Cmd).Run()
      GOROOT/src/os/exec/exec.go:587 +0x26
  github.com/bazelbuild/rules_go/go/tools/bzltestutil.Wrap()
      external/io_bazel_rules_go/go/tools/bzltestutil/wrap.go:143 +0xda4
  main.main()
      bazel-out/k8-fastbuild/bin/pkg/store/mockstore/unistore/tikv/tikv_test_/testmain.go:131 +0x47
Goroutine 53 (running) created at:
  github.com/bazelbuild/rules_go/go/tools/bzltestutil.Wrap()
      external/io_bazel_rules_go/go/tools/bzltestutil/wrap.go:131 +0xd8a
  main.main()
      bazel-out/k8-fastbuild/bin/pkg/store/mockstore/unistore/tikv/tikv_test_/testmain.go:131 +0x47 

4. What is your TiDB version? (Required)

@hawkingrei
Copy link
Member Author

We can fix the problem by upgrading the rules_go.

@hawkingrei hawkingrei added the good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines. label Jan 9, 2024
ti-chi-bot bot pushed a commit that referenced this issue Jan 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/test good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines. severity/moderate type/bug The issue is confirmed as a bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant