Skip to content

Commit 12cbb18

Browse files
committed
[improve][broker] Avoid PersistentReplicator.expireMessages logic compute backlog twice (#23957)
1 parent 461a9a6 commit 12cbb18

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentReplicator.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -670,8 +670,8 @@ public void updateMessageTTL(int messageTTLInSeconds) {
670670

671671
@Override
672672
public boolean expireMessages(int messageTTLInSeconds) {
673-
if ((cursor.getNumberOfEntriesInBacklog(false) == 0)
674-
|| (cursor.getNumberOfEntriesInBacklog(false) < MINIMUM_BACKLOG_FOR_EXPIRY_CHECK
673+
long backlog = cursor.getNumberOfEntriesInBacklog(false);
674+
if ((backlog == 0) || (backlog < MINIMUM_BACKLOG_FOR_EXPIRY_CHECK
675675
&& !topic.isOldestMessageExpired(cursor, messageTTLInSeconds))) {
676676
// don't do anything for almost caught-up connected subscriptions
677677
return false;

0 commit comments

Comments
 (0)