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


   @itantiger The priority of loading props is a bit chaos in this case. It 
seems to me the property of loading properties is shown below.
   1. command-lind
   1. config file (consumer.config)
   1. default value
   
   The current priority is shown below.
   1. command-lind
   1. default value
   1. config file (consumer.config)
   
   The priority from this PR is shown below.
   1. config file (consumer.config)
   1. command-lind
   1. default value
   
   
   Of course, current priority has issue as properties from config file always 
get overwritten by default value. We need to fix it. However, I'd like to check 
the priority again. WDYT? Should config file has higher priority than 
command-lind?


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