showuon opened a new pull request #11845:
URL: https://github.com/apache/kafka/pull/11845


   1. Remove `decommissioning brokers`  section
   When reading `decommissioning brokers` section in document, it makes me 
confused:
   
![image](https://user-images.githubusercontent.com/43372967/156732442-3e8452dd-fc7c-419a-9330-58b141cf2ccd.png)
   
   > The partition reassignment tool does not have the ability to automatically 
generate a reassignment plan for decommissioning brokers yet.
   
   We should be able to auto generate a reassignment plan for decommissioning 
brokers via `--generate` argument
   > the reassignment needs to ensure that all the replicas are not moved from 
the decommissioned broker to only one other broker.
   
   We already can move the replicas evenly across all the brokers
   > To make this process effortless, we plan to add tooling support for 
decommissioning brokers in the future. 
   
   This section is written in 2015, and I don't think we still have plan to 
support it (or we already supported)
   
   Remove the section.
   
   2. Update the `--generate` replicas plan with current version.
   
   Also, the example in the `--generate` command, it assign all the partition 
leader to the same one (broker 5), which will confuse users to think this tool 
is unreliable. After running a test, I confirmed we can generate a plan evenly 
assigning the replicas across the brokers. Update it.
   
![image](https://user-images.githubusercontent.com/43372967/156733715-52c118cd-2269-4405-8cda-f3c879ca0155.png)
   
   3. Update the `throttle` updated command by adding `--additional`
   After KIP-455 and this [PR](https://github.com/apache/kafka/pull/8891/), we 
have to provide `--additional` flag to allow change the throttle config. 
   
   4. Update the reassign-partition script output with current version
   
   
   ### 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