zzccctv commented on pull request #11496:
URL: https://github.com/apache/kafka/pull/11496#issuecomment-972630217


   @guozhangwang This suggestion looks very good, But there is a problem. The 
BrokerConfigHandler class also calls the updateDefaultConfig and 
updateBrrokerConfig functions respectively. It simply adds the validateonly 
parameter to updateCurrentConfig and sets false in the updateDefaultConfig 
function. Although it solves the problem of repeatedly printing logs during 
startup, during dynamic configuration adjustment, It seems that this function 
will not work. Can we add a parameter to the processReconfiguration function to 
control the print log?


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