dengziming opened a new pull request #11522:
URL: https://github.com/apache/kafka/pull/11522


   *More detailed description of your change*
   This patch fixes several problems with the UpdateFeatures API in KRaft:
   
   KafkaApis did not properly forward this request type to the controller.
   ControllerApis did not handle the request type.
   Some other small fixes.
   
   *Summary of testing strategy (including rationale)*
   1. unit test in FeatureControlManagerTest
   2. I tried to converted UpdateFeaturesTest to use KRaft, but I encountered 
some problems, for example, we use totally different class in KRaft and zk 
mode, and we can overwrite data in zk path but we can't overwrite data in KRaft 
metadata topic, so I decided to handle this in a future PR.
   
   ### Committer Checklist (excluded from commit message)
   - [ ] Verify design and implementation 
   - [ ] Verify test coverage and CI build status
   - [ ] Verify documentation (including upgrade notes)
   


-- 
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


Reply via email to