kkonstantine commented on pull request #8944:
URL: https://github.com/apache/kafka/pull/8944#issuecomment-653631904


   What I don't like too much about this test is that it fails on optional 
configs. But on the other hand I guess it's good to have a test for a complete 
listing, it's just that this is failing after a merge. Thanks for the fix. I 
confirmed by running this system test locally. Unit/integration tests are not 
relevant here. 
   
   


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

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


Reply via email to