yuruguo opened a new pull request #12936:
URL: https://github.com/apache/pulsar/pull/12936


   ### Motivation
   Currently, the deletion topic method for `v1/topic` has the problem of 
parameter loss and mismatch as below,
   1. `deleteSchema` is missing
   
https://github.com/apache/pulsar/blob/af1a82d1aaf7965d3080a6d043d5111f0b9015e5/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/v1/PersistentTopics.java#L302-L308
   
https://github.com/apache/pulsar/blob/af1a82d1aaf7965d3080a6d043d5111f0b9015e5/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/v1/PersistentTopics.java#L263-L270
   2. `force` and `deleteSchema` are confused
   
https://github.com/apache/pulsar/blob/af1a82d1aaf7965d3080a6d043d5111f0b9015e5/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/v1/PersistentTopics.java#L307
   
https://github.com/apache/pulsar/blob/af1a82d1aaf7965d3080a6d043d5111f0b9015e5/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/PersistentTopicsBase.java#L1013-L1018
   
   ### Modifications
   Class: `v1/PersistentTopics`
   - Add @QueryParam `deleteSchema` in method `deleteTopic` and 
`deletePartitionedTopic`
   - Correct parameter `force` and `deleteSchema` in method `deleteTopic`
   
   ### Documentation  
   - [x] `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