eduwercamacaro commented on PR #20709:
URL: https://github.com/apache/kafka/pull/20709#issuecomment-3433190226

   Hi @Nikita-Shupletsov 
   I made a new PR (https://github.com/apache/kafka/pull/20749). In that case, 
the PR works more like a proposal than an experiment. Since I made this PR just 
as an experiment, it has too many changes that aren't needed. I’m very sorry to 
move this conversation. 
   
   The integration test that you mentioned was extremely useful for me to 
understand the problem. I copy-pasted your test in this branch and the test 
passes. [check this 
change](https://github.com/apache/kafka/pull/20749/files#diff-a8f919d43ec1cebdb136acf94adfbeb20fef0cda0ad5d29b7010e0186b355d71R300)
   
   Can you please take a look on that PR? 
   
   Thanks for your help on this. I'm trying to explore all the options to 
implement KIP-1035 and I believe my proposal also solves KAFKA-19434.


-- 
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]

Reply via email to