poorbarcode opened a new pull request, #24622:
URL: https://github.com/apache/pulsar/pull/24622

   ### Motivation
   
   **Background**: the steps of expiry of a topic
   - Call expiry for each subscription
     - The subscription finds the target position, then calls mark deleting for 
the cursor
   - Call expiry for each replicator
     - The replicator finds the target position, then calls mark deleting for 
the cursor
   
   So the task calls for finding target position many times when a topic 
expires. But the finding position operation is really expensive
   
   ### Modifications
   
   Since all of the tasks that find the target position that relates to the 
same topic will get the same result, we can call the finding operation at most 
once, during expiring messages for a topic
   
   ### Documentation
   
   <!-- DO NOT REMOVE THIS SECTION. CHECK THE PROPER BOX ONLY. -->
   
   - [ ] `doc` <!-- Your PR contains doc changes. -->
   - [ ] `doc-required` <!-- Your PR changes impact docs and you will update 
later -->
   - [x] `doc-not-needed` <!-- Your PR changes do not impact docs -->
   - [ ] `doc-complete` <!-- Docs have been already added -->
   
   ### Matching PR in forked repository
   
   PR in forked repository: x


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