[GitHub] [kafka] artemlivshits commented on pull request #12462: [KAFKA-9965] Fix accumulator tryAppend, so that fresh new producerBatch is created

["artemlivshits (via GitHub)" <gi...@apache.org>]

artemlivshits commented on PR #12462:
URL: https://github.com/apache/kafka/pull/12462#issuecomment-1550270236

   I think it's still a choice between breaking old default sticky partitioner logic vs. not fixing even older round robin partitioner logic (which has been broken for a few years) vs. using reflection to sort of eat the cake and have it too.  One thing that might've changed with time is that if people got confidence in the new built-in partitioner logic, maybe it's ok to break the old default sticky partitioner, but it's hard to tell.


-- 
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: jira-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org