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 at DDLSeqNum #44470

Closed
Tracked by #41316
xhebox opened this issue Jun 7, 2023 · 1 comment
Closed
Tracked by #41316

DATA RACE at DDLSeqNum #44470

xhebox opened this issue Jun 7, 2023 · 1 comment
Labels
component/test sig/sql-infra SIG: SQL Infra type/bug The issue is confirmed as a bug.

Comments

@xhebox
Copy link
Contributor

xhebox commented Jun 7, 2023

Bug Report

Looks like another bug like #32541. /cc @hawkingrei

1. Minimal reproduce step (Required)

https://do.pingcap.net/jenkins/blue/organizations/jenkins/pingcap%2Ftidb%2Fghpr_mysql_test/detail/ghpr_mysql_test/23001/pipeline/192

WARNING: DATA RACE

Read at 0x00c000d83b98 by goroutine 514:

  github.com/pingcap/tidb/ddl.(*worker).writeDDLSeqNum()

      /home/jenkins/agent/workspace/pingcap/tidb/ghpr_mysql_test/tidb/ddl/ddl_worker.go:617 +0x88

  github.com/pingcap/tidb/ddl.(*worker).finishDDLJob()

      /home/jenkins/agent/workspace/pingcap/tidb/ghpr_mysql_test/tidb/ddl/ddl_worker.go:609 +0x4bb

  github.com/pingcap/tidb/ddl.(*worker).HandleJobDone()

      /home/jenkins/agent/workspace/pingcap/tidb/ghpr_mysql_test/tidb/ddl/ddl_worker.go:707 +0x49

  github.com/pingcap/tidb/ddl.(*worker).HandleDDLJobTable()

      /home/jenkins/agent/workspace/pingcap/tidb/ghpr_mysql_test/tidb/ddl/ddl_worker.go:761 +0x490

  github.com/pingcap/tidb/ddl.(*ddl).delivery2worker.func1()

      /home/jenkins/agent/workspace/pingcap/tidb/ghpr_mysql_test/tidb/ddl/job_table.go:420 +0xb58

  github.com/pingcap/tidb/util.(*WaitGroupWrapper).Run.func1()

      /home/jenkins/agent/workspace/pingcap/tidb/ghpr_mysql_test/tidb/util/wait_group_wrapper.go:154 +0x73


Previous write at 0x00c000d83b98 by goroutine 353:

  github.com/pingcap/tidb/ddl.(*ddl).Start.func1()

      /home/jenkins/agent/workspace/pingcap/tidb/ghpr_mysql_test/tidb/ddl/ddl.go:758 +0x86

  github.com/pingcap/tidb/owner.(*mockManager).toBeOwner()

      /home/jenkins/agent/workspace/pingcap/tidb/ghpr_mysql_test/tidb/owner/mock.go:82 +0x36e

  github.com/pingcap/tidb/owner.(*mockManager).CampaignOwner.func1()

      /home/jenkins/agent/workspace/pingcap/tidb/ghpr_mysql_test/tidb/owner/mock.go:136 +0x511

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

3. What did you see instead (Required)

4. What is your TiDB version? (Required)

@xhebox xhebox added type/bug The issue is confirmed as a bug. sig/sql-infra SIG: SQL Infra labels Jun 7, 2023
@Defined2014
Copy link
Contributor

fixed by #44536

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/test sig/sql-infra SIG: SQL Infra type/bug The issue is confirmed as a bug.
Projects
None yet
Development

No branches or pull requests

3 participants