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]