chia7712 commented on PR #15417:
URL: https://github.com/apache/kafka/pull/15417#issuecomment-2124336401

   > If we postpone rewriting ConfigCommand itself, please, tell me, if we 
going to proceed with the https://github.com/apache/kafka/pull/15848 and merge 
remaining ConfigCommand tests?
   
   I prefer to complete tests migration for following reasons.
   
   1. migrating tests code have less impaction to production code
   2. we are increasing the test coverage for those command tools. By migration 
we can avoid adding more and more scala code
   
   At any rate, @nizhikov you have complete many great migration for kafka. We 
can postpone rewriting for a while (especially, 3.8.0 is going to be released). 
Meanwhile, it would be great to take a look at flaky if @nizhikov you have free 
time. That will be another great contribution to Kafka as those flaky [0] can 
imply something wrong in our code base. WDYT? I'm working with other 
contributors to fix those flaky, and it would be nice to have you in this party 
:)
   
   [0] 
https://issues.apache.org/jira/issues/?jql=project%20%3D%20KAFKA%20AND%20resolution%20%3D%20Unresolved%20AND%20text%20~%20%22flaky%22%20ORDER%20BY%20priority%20DESC%2C%20updated%20DESC
   
   


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

Reply via email to