chia7712 commented on pull request #9507:
URL: https://github.com/apache/kafka/pull/9507#issuecomment-717021604


   @showuon Thanks for your patch. Could you delete unnecessary parameters 
(BOOTSTRAP_SERVERS_CONFIG and APPLICATION_ID_CONFIG) from the tests which are 
using ```TopologyTestDriver``` (please notice below comment I copied from 
KAFKA-10628).
   
   > This task is to track down all those unnecessary parameters and delete 
them. You can consult the above pull request for some examples of this.
   > Note that there are times when the application id is actually significant, 
since it is used in conjunction with the state directory to give the driver a 
unique place to store local state. On the other hand, it would be sufficient to 
just set a unique state directory and not bother with the app id in that case.


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