rdhabalia commented on a change in pull request #13073:
URL: https://github.com/apache/pulsar/pull/13073#discussion_r803185954



##########
File path: 
pulsar-broker/src/main/java/org/apache/pulsar/broker/service/PulsarStats.java
##########
@@ -139,6 +139,8 @@ public synchronized void updateStats(
                                 // this task: helps to activate 
inactive-backlog-cursors which have caught up and
                                 // connected, also deactivate 
active-backlog-cursors which has backlog
                                 topic.checkBackloggedCursors();
+                                // check if topic is inactive and require 
ledger rollover
+                                ((PersistentTopic) 
topic).checkInactiveLedgers();

Review comment:
       no, it doesn't depend on any feature. this background thread is being 
used for multiple simple topic checks such as backlogged-cursor or 
inactive-ledger checks. and it's internal implementation and we don't have to 
document it explicitly because it's just a safeguard check at the broker.




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