rdhabalia commented on a change in pull request #4066: Allow to configure the 
managed ledger cache eviction frequency
URL: https://github.com/apache/pulsar/pull/4066#discussion_r278286178
 
 

 ##########
 File path: 
pulsar-broker/src/main/java/org/apache/pulsar/broker/service/PulsarStats.java
 ##########
 @@ -136,9 +136,6 @@ public synchronized void updateStats(
                                 } catch (Exception e) {
                                     log.error("Failed to generate topic stats 
for topic {}: {}", name, e.getMessage(), e);
                                 }
-                                // this task: helps to activate 
inactive-backlog-cursors which have caught up and
-                                // connected, also deactivate 
active-backlog-cursors which has backlog
-                                ((PersistentTopic) 
topic).getManagedLedger().checkBackloggedCursors();
 
 Review comment:
   if we have removed from here then who triggers `checkBackloggedCursors(..)`?

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to