cockroachdb / replicator

replicator is a toolkit for ingesting logical replication feeds into a CockroachDB cluster

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Data race in leases_test.go

bobvawter opened this issue · comments

https://github.com/cockroachdb/replicator/actions/runs/9070784347/job/24923391620#step:7:6779

==================
WARNING: DATA RACE
Write at 0x00c00043c120 by goroutine 109:
  github.com/cockroachdb/replicator/internal/staging/leases.TestLeases.func7.1()
      /home/runner/work/replicator/replicator/internal/staging/leases/leases_test.go:252 +0x3a
  runtime.deferreturn()
      /opt/hostedtoolcache/go/1.22.2/x64/src/runtime/panic.go:602 +0x5d
  testing.tRunner()
      /opt/hostedtoolcache/go/1.22.2/x64/src/testing/testing.go:1689 +0x21e
  testing.(*T).Run.gowrap1()
      /opt/hostedtoolcache/go/1.22.2/x64/src/testing/testing.go:1742 +0x44

Previous read at 0x00c00043c120 by goroutine 6:
  github.com/cockroachdb/replicator/internal/staging/leases.(*leases).tryRelease()
      /home/runner/work/replicator/replicator/internal/staging/leases/leases.go:437 +0x197
  github.com/cockroachdb/replicator/internal/staging/leases.(*leases).release.func1()
      /home/runner/work/replicator/replicator/internal/staging/leases/leases.go:423 +0xe4
  github.com/cockroachdb/replicator/internal/staging/leases.(*leases).release.Retry.func2()
      /home/runner/work/replicator/replicator/internal/util/retry/retry.go:69 +0x4f
  github.com/cockroachdb/replicator/internal/util/retry.Loop()
      /home/runner/work/replicator/replicator/internal/util/retry/retry.go:98 +0x213
  github.com/cockroachdb/replicator/internal/util/retry.Retry()
      /home/runner/work/replicator/replicator/internal/util/retry/retry.go:68 +0x21a
  github.com/cockroachdb/replicator/internal/staging/leases.(*leases).release()
      /home/runner/work/replicator/replicator/internal/staging/leases/leases.go:421 +0x95
  github.com/cockroachdb/replicator/internal/staging/leases.(*leases).Acquire.func2()
      /home/runner/work/replicator/replicator/internal/staging/leases/leases.go:179 +0xe4
  github.com/cockroachdb/replicator/internal/staging/leases.(*leaseFacade).Release()
      /home/runner/work/replicator/replicator/internal/staging/leases/leases.go:113 +0xa1
  github.com/cockroachdb/replicator/internal/staging/leases.(*leases).Acquire.func3()
      /home/runner/work/replicator/replicator/internal/staging/leases/leases.go:185 +0x12

Goroutine 109 (running) created at:
  testing.(*T).Run()
      /opt/hostedtoolcache/go/1.22.2/x64/src/testing/testing.go:1742 +0x825
  github.com/cockroachdb/replicator/internal/staging/leases.TestLeases()
      /home/runner/work/replicator/replicator/internal/staging/leases/leases_test.go:243 +0xbe9
  testing.tRunner()
      /opt/hostedtoolcache/go/1.22.2/x64/src/testing/testing.go:1689 +0x21e
  testing.(*T).Run.gowrap1()
      /opt/hostedtoolcache/go/1.22.2/x64/src/testing/testing.go:1742 +0x44

Goroutine 6 (running) created at:
  runtime.createfing()
      /opt/hostedtoolcache/go/1.22.2/x64/src/runtime/mfinal.go:164 +0x3c
  os.NewFile()
      /opt/hostedtoolcache/go/1.22.2/x64/src/os/file_unix.go:119 +0x87
  os.init()
      /opt/hostedtoolcache/go/1.22.2/x64/src/os/file.go:65 +0x332
==================