urbandan commented on PR #10566: URL: https://github.com/apache/kafka/pull/10566#issuecomment-1458499533
@C0urante Thanks for the prototype, went through the change and left a few comments. Overall, I think your solution would solve this issue as well. On the other hand, I still think that the current API (even with your solution included) is still ambiguous, and the mix of mutable and immutable data will cause some tricky bugs in dependent code bases. Still, your solution would be a nice improvement compared to the current situation, and I don't expect any API changes going through anytime soon :) -- 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-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
