aloyszhang opened a new pull request #14655:
URL: https://github.com/apache/pulsar/pull/14655


   
   Fixes #14654 
   
   ### Motivation
   Currently, pulsar admin command for schemas can't delete a schema 
completely, it only appends a deleted  `schemaInfo` to mark this schema as 
deleted.
   This may lead to some problems as described in #14654 
   This pull request aims to delete schema forcefully which will remove the 
schema from the metastore and bookkeeper completely.
   
   
   ### Modifications
   1. Add a new option for `DeleteSchema` to indicate delete schema forcefully 
or not, default is false
   2. If delete schema forcefully, remove it from metastore and bookkeeper
   3. Add test for force delete schema in 
`SchemaDeleteTest.schemaForceDeleteTest`
   
   ### Verifying this change
   
   
   ### 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): (yes / no)
     - The public API: (yes / no)
     - The schema: (yes / no / don't know)
     - The default values of configurations: (yes / no)
     - The wire protocol: (yes / no)
     - The rest endpoints: (yes / no)
     - The admin cli options: (yes / no)
     - Anything that affects deployment: (yes / no / don't know)
   
   ### Documentation
   
   Check the box below or label this PR directly (if you have committer 
privilege).
   
     
   - [ ] `no-need-doc` 
     
   
   
   
   


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