[Broker] Call .release() when discarding entry to prevent direct memo…

…ry leak (apache#11748)
datastax · Aug 23, 2021 · 7906bb5 · 7906bb5
1 parent 8027ab4
commit 7906bb5
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/...org/apache/pulsar/broker/service/persistent/PersistentDispatcherSingleActiveConsumer.java b/...org/apache/pulsar/broker/service/persistent/PersistentDispatcherSingleActiveConsumer.java
@@ -175,7 +175,9 @@ public synchronized void internalReadEntriesComplete(final List<Entry> entries,
                 Entry entry = iterator.next();
                 byte[] key = peekStickyKey(entry.getDataBuffer());
                 Consumer consumer = stickyKeyConsumerSelector.select(key);
+                // Skip the entry if it's not for current active consumer.
                 if (consumer == null || currentConsumer != consumer) {
+                    entry.release();
                     iterator.remove();
                 }
             }