mimaison commented on PR #16665: URL: https://github.com/apache/kafka/pull/16665#issuecomment-2276206257
So is this the format we're going for? I see we have a constructor accepting `AbstractConfig` and getters for all fields, plus potentially a `validate()` method for any validations that only rely on configs in the class. I think it would be good to document the expected target and explain the pros and cons vs alternatives in the Jira. -- 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
