dajac commented on PR #18144: URL: https://github.com/apache/kafka/pull/18144#issuecomment-2541498458
@squah-confluent Thanks for the patch. I wonder if we could approach it a bit differently though. My understanding of the issue is that we left an empty batch and it got flushed after the linger time without having any records. I think that having an empty batch is not wrong per say so I wonder if we could update the flush method to ignore empty batches. If we do this, we should also update at L928 to only add the current event to the deferred events of the batch only if the batch is not null and not empty. Would it work? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
