cmccabe commented on PR #17952: URL: https://github.com/apache/kafka/pull/17952#issuecomment-2524568957
Thanks for the PR. Thoughts: I think feature control should not have a reference to cluster control. in general many things reference feature control to know the MV and I don’t want them to have to pull in cluster control for that. It would probably be better to have ConfigurationControl handle the updateFeatures call, and have it call into FeatureControl. -- 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
