showuon commented on a change in pull request #11800:
URL: https://github.com/apache/kafka/pull/11800#discussion_r820324135
##########
File path:
clients/src/main/java/org/apache/kafka/common/config/AbstractConfig.java
##########
@@ -376,11 +384,24 @@ private void logAll() {
log.info(b.toString());
}
+ public void logUnusedAndUnknown() {
Review comment:
TBH, no, I've never considered that! Thanks for the reminder.
@RivenSun2 , sorry, I forgot the `AbstractConfig` is a public API. But since
the v3.2.0 KIP freeze has passed, and I think this KIP might need more
discussion, could you submit another PR to revert this change first? After KIP
discussion/voting passed, we can start to work on it again. If you don't have
time to submit a revert PR within this week, please let me know. 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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
