storage/spanlatch: pass snapshot by reference to wait

This does not escape. Release note: None
cockroachdb · Dec 26, 2018 · d1dfefc · d1dfefc
1 parent c7e0fef
commit d1dfefc
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/pkg/storage/spanlatch/manager.go b/pkg/storage/spanlatch/manager.go
@@ -208,7 +208,7 @@ func (m *Manager) Acquire(
 	lg, snap := m.sequence(spans, ts)
 	defer snap.close()
 
-	err := m.wait(ctx, lg, snap)
+	err := m.wait(ctx, lg, &snap)
 	if err != nil {
 		m.Release(lg)
 		return nil, err
@@ -335,7 +335,7 @@ func ifGlobal(ts hlc.Timestamp, s spanset.SpanScope) hlc.Timestamp {
 
 // wait waits for all interfering latches in the provided snapshot to complete
 // before returning.
-func (m *Manager) wait(ctx context.Context, lg *Guard, snap snapshot) error {
+func (m *Manager) wait(ctx context.Context, lg *Guard, snap *snapshot) error {
 	timer := timeutil.NewTimer()
 	timer.Reset(base.SlowRequestThreshold)
 	defer timer.Stop()

diff --git a/pkg/storage/spanlatch/manager_test.go b/pkg/storage/spanlatch/manager_test.go
@@ -104,7 +104,7 @@ func (m *Manager) MustAcquireChCtx(
 	ch := make(chan *Guard)
 	lg, snap := m.sequence(spans, ts)
 	go func() {
-		err := m.wait(ctx, lg, snap)
+		err := m.wait(ctx, lg, &snap)
 		if err != nil {
 			m.Release(lg)
 			lg = nil