wuzhanpeng opened a new pull request, #15301:
URL: https://github.com/apache/pulsar/pull/15301

   
   ### Motivation
   
   Currently executing the command `bin/pulsar 
initialize-transaction-coordinator-metadata` will directly update the number of 
partitions of `persistent://pulsar/system/transaction_coordinator_assign`.
   
   When accidentally misoperation, the number of partitions is directly 
modified, and there is no way to roll back (the number of partitions cannot be 
reduced).
   
   ### Modifications
   
   Simplely add an argument to determine whether forced update is allowed.
   
   ### Verifying this change
   
   This change added tests and can be verified as follows:
   
   *(example:)*
   ```shell
   bin/pulsar initialize-transaction-coordinator-metadata -cs 127.0.0.1:2181 -c 
standalone -c 1
   bin/pulsar initialize-transaction-coordinator-metadata -cs 127.0.0.1:2181 -c 
standalone -c 2
   
   # Check for changes of partitions
   bin/pulsar-admin topics get-partitioned-topic-metadata 
persistent://pulsar/system/transaction_coordinator_assign
   ```
   
   ### Does this pull request potentially affect one of the following parts:
   
   *If `yes` was chosen, please highlight the changes*
   
     - Dependencies (does it add or upgrade a dependency): (no)
     - The public API: (no)
     - The schema: (no)
     - The default values of configurations: (no)
     - The wire protocol: (no)
     - The rest endpoints: (no)
     - **The admin cli options: (yes)**
     - Anything that affects deployment: (no)
   
   ### Documentation
   
   Check the box below or label this PR directly.
   
   Need to update docs? 
     
   - [x] `no-need-doc` 
   I think command description is sufficient.


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

Reply via email to