[GitHub] [kafka] mimaison commented on pull request #9313: [mm2] Fix consumer/producer properties override

2020-10-16 Thread GitBox
mimaison commented on pull request #9313: URL: https://github.com/apache/kafka/pull/9313#issuecomment-710160841 Test failures are not related, merging to trunk This is an automated message from the Apache Git Service. To

[GitHub] [kafka] mimaison commented on pull request #9313: [mm2] Fix consumer/producer properties override

2020-10-15 Thread GitBox
mimaison commented on pull request #9313: URL: https://github.com/apache/kafka/pull/9313#issuecomment-709415966 @scanterog I've rekicked the build This is an automated message from the Apache Git Service. To respond to the

[GitHub] [kafka] mimaison commented on pull request #9313: [mm2] Fix consumer/producer properties override

2020-10-09 Thread GitBox
mimaison commented on pull request #9313: URL: https://github.com/apache/kafka/pull/9313#issuecomment-705563795 @scanterog `testClusterConfigProperties()` seems to be failing now. Can you fix it before I take a look at the PR? ```

[GitHub] [kafka] mimaison commented on pull request #9313: [mm2] Fix consumer/producer properties override

2020-10-08 Thread GitBox
mimaison commented on pull request #9313: URL: https://github.com/apache/kafka/pull/9313#issuecomment-705563795 @scanterog `testClusterConfigProperties()` seems to be failing now. Can you fix it before I take a look at the PR? ```

[GitHub] [kafka] mimaison commented on pull request #9313: [mm2] Fix consumer/producer properties override

2020-09-25 Thread GitBox
mimaison commented on pull request #9313: URL: https://github.com/apache/kafka/pull/9313#issuecomment-698229647 @scanterog Thanks for the PR. Can you add a test for the changes? This is an automated message from the Apache

[GitHub] [kafka] mimaison commented on pull request #9313: [mm2] Fix consumer/producer properties override

2020-09-24 Thread GitBox
mimaison commented on pull request #9313: URL: https://github.com/apache/kafka/pull/9313#issuecomment-698229647 @scanterog Thanks for the PR. Can you add a test for the changes? This is an automated message from the Apache