antimpatel commented on issue #5905: [pulsar-broker][REST API] Support `terminate` all partitions of a partitioned topic URL: https://github.com/apache/pulsar/issues/5905#issuecomment-576964837 @zymap Since terminate Api returns the messageId, so To accommodate the above changes we also need to change the response of the API if we want to implement this in the same method. Why it's not recommended by you to create a new method in API and in TopicBase to accommodate this change? I believe it's better to have two separate API method, one for a single topic and other for the partitioned topic, because it's more readable and abstracted out as per API description.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
