From 85daa34683d03d39ead23e112946d66c86b67185 Mon Sep 17 00:00:00 2001 From: Rajan Dhabalia Date: Thu, 20 Feb 2025 03:28:17 -0800 Subject: [PATCH] [fix][broker] Fix NPE during active consumer is disconnected (#24008) (cherry picked from commit dbc09e14f9439b58b6e4884cf82f3a6081662445) --- .../persistent/PersistentDispatcherSingleActiveConsumer.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentDispatcherSingleActiveConsumer.java b/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentDispatcherSingleActiveConsumer.java index e6a9b9c910e41..a67b6829fe6b0 100644 --- a/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentDispatcherSingleActiveConsumer.java +++ b/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentDispatcherSingleActiveConsumer.java @@ -350,7 +350,8 @@ void readMoreEntries(Consumer consumer) { final Consumer activeConsumer = getActiveConsumer(); if (consumer != activeConsumer) { log.info("[{}] cancel the readMoreEntries because consumer {} is no longer the active consumer {}", - topic.getName(), consumer.consumerName(), activeConsumer.consumerName()); + topic.getName(), consumer.consumerName(), + activeConsumer != null ? activeConsumer.consumerName() : null); return; } if (havePendingRead) {