You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/10/19 16:59:47 UTC

[GitHub] [pulsar] pgier opened a new issue, #18119: Sending "stop" to a connector should be idempotent.

pgier opened a new issue, #18119:
URL: https://github.com/apache/pulsar/issues/18119

   ### Search before asking
   
   - [X] I searched in the [issues](https://github.com/apache/pulsar/issues) and found nothing similar.
   
   
   ### Motivation
   
   We're running pulsar in Kubernetes and ran into a situation where the "stop" command was called on a connector (a sink in this case) but the kubernetes statefulset was not removed.  Calling "stop" again on the same connector returned an HTTP 400 error with "operation not permitted" because pulsar thought that the connector was already stopped.
   
   I was able to work around this by first calling "start" on the connector, either from the pulsar-admin CLI or the rest endpoint, and then calling "stop" again.  This time the stateful set was removed.
   
   ### Solution
   
   Instead of returning different responses (HTTP 200 vs. 400) depending on the state of the connector, I think it would be better if a "stop" call on an already stopped connector would try again to stop the connector and clean up any hanging resources, and then return a 200 if everything went ok.  So several calls to stop on a connector in a row should all return HTTP 200 assuming that the connector was stopped successfully.
   
   ### Alternatives
   
   _No response_
   
   ### Anything else?
   
   _No response_
   
   ### Are you willing to submit a PR?
   
   - [ ] I'm willing to submit a PR!


-- 
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: commits-unsubscribe@pulsar.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar] github-actions[bot] commented on issue #18119: Sending "stop" to a connector should be idempotent.

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on issue #18119:
URL: https://github.com/apache/pulsar/issues/18119#issuecomment-1320751022

   The issue had no activity for 30 days, mark with Stale label.


-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org