codelipenghui commented on PR #15757:
URL: https://github.com/apache/pulsar/pull/15757#issuecomment-1136921453

   > Honestly I don't like that much that behavior and I would prefer to let 
the user modify the properties using an explicit PulsarAdmin function (see 
https://github.com/apache/pulsar/pull/15751 )
   
   @codelipenghui do you know why we allowed to override the properties from 
the first Consumer?
   
   Hmm, I'm thinking more about it. The correct behavior should not be the 
first consumer override the properties, it should be "if the subscription 
absents, the first consumer will trigger the subscription creation, so the 
subscription will create with the properties that the first consumer has". The 
initial position also follows this way.
   
   So it should be the initial implementation of PIP-105 missed to persist the 
properties, causing the "first consumer override properties"


-- 
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-unsubscr...@pulsar.apache.org

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

Reply via email to