sink w/ no acks: debug log needs to be before finishBatch

twmb · Apr 28, 2021 · 43a0009 · 43a0009
1 parent 948ce3e
commit 43a0009
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/pkg/kgo/sink.go b/pkg/kgo/sink.go
@@ -549,10 +549,10 @@ func (s *sink) handleReqResp(req *produceRequest, resp kmsg.Response, err error)
 			for partition, batch := range partitions {
 				batch.owner.mu.Lock()
 				if batch.isOwnersFirstBatch() {
-					s.cl.finishBatch(batch.recBatch, req.producerID, req.producerEpoch, partition, 0, nil)
 					if debug {
-						fmt.Fprintf(b, "%d{%d=>%d}, ", partition, 0, len(batch.records))
+						fmt.Fprintf(b, "%d{0=>%d}, ", partition, len(batch.records))
 					}
+					s.cl.finishBatch(batch.recBatch, req.producerID, req.producerEpoch, partition, 0, nil)
 				} else if debug {
 					fmt.Fprintf(b, "%d{skipped}, ", partition)
 				}