Free the lock before adding to channel (treeverse#1391)

itaiad200 · web-flow · commit f864e32eb257 · 2021-02-01T20:41:13.000+02:00
diff --git a/graveler/committed/batch.go b/graveler/committed/batch.go
@@ -43,15 +43,16 @@ var (
 // If Wait() has already been called, returns an error.
 func (bc *BatchCloser) CloseWriterAsync(w ResultCloser) error {
 	bc.mu.Lock()
-	defer bc.mu.Unlock()
 
 	if bc.err != nil {
 		// Don't accept new writers if previous error occurred.
 		// In particular, if Wait has started then this is errMultipleWaitCalls.
+		bc.mu.Unlock()
 		return bc.err
 	}
 
 	bc.wg.Add(1)
+	bc.mu.Unlock()
 	bc.ch <- w
 
 	return nil
