showuon edited a comment on pull request #10811: URL: https://github.com/apache/kafka/pull/10811#issuecomment-877736694
@rondagostino , thanks for comments and many good catch! I've updated the PR. Also update the PR description. > The PR as currently written restricts describing configs via --zookeeper to user and broker configs. Unlike with altering, it does not perform a check to see if the broker is not started. Do we wish to add such a check? Good suggestion! Make sense to me. I added the check for broker type describe via `--zookeeper`, and add tests. Thank you. -- 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