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

   > lastActive is set to clock.millis() while initilized, but will be covered 
by recover() soon.
   
   I see.
   
   Actually, only ledger rollover and updating the cursor Ledger failed will 
cause the `lastActive` update to zookeeper right?
   It also has a chance to delete the subscription unexpectedly if no ledger 
rollover and cursor ledger update failure happens.


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