rdhabalia opened a new pull request #2269: handle subscription-already-exist exception on partitioned-topic for create-sub admin-api URL: https://github.com/apache/incubator-pulsar/pull/2269 ### Motivation right now, application uses admin api to create subscription on partitioned topic. if subscription creation fails for one of the partition then it gives error to application. so, if application retries to create subscription again then admin api always gives `subscription-already-exist` exception (because previous call has partially created subscription on some of the partitions) and user/application will never be able to handle this scenario correctly. ### Modifications handle partially failed request for create-subscription api. admin api fails request only if - subscription is created for all partitions ### Result server handles and sends correct error-response for partially failed create subscription request.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on 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
