0xffff-zhiyan commented on PR #21053: URL: https://github.com/apache/kafka/pull/21053#issuecomment-3729936703
@ahuang98 Thanks! I looked into `validateAlterConfig`, and the flow works like this: when we alter configs, we first read all existing configs from the snapshot into a map. Then we apply the config-altered records to update the configs in that map, and finally we validate the entire map. Because of this, it’s hard to cleanly separate validation of existing configs from validation of new configs. So I added a filter when reading existing configs from the snapshot to ensure that all configs in the map are valid. -- 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]
