codelipenghui commented on pull request #13949:
URL: https://github.com/apache/pulsar/pull/13949#issuecomment-1023810352


   @michaeljmarshall Thanks for the explanation, LGTM. 
   
   > Although, technically, if the producer is already created, we just respond 
that it was created successfully. I am not sure that I like this design, but 
that is a different discussion.
   
   Yes, it should be a different discussion, I think we should return such as a 
`ProducerConflict` exception to the client, otherwise, if the client 
implementation does not follow the spec, it will have more than 1 active 
producer instance with the same producer ID.


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