BewareMyPower commented on code in PR #22201:
URL: https://github.com/apache/pulsar/pull/22201#discussion_r1513919792


##########
pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerImpl.java:
##########
@@ -2279,14 +2287,25 @@ private void seekAsyncInternal(long requestId, ByteBuf 
seek, MessageId seekId, S
                 lastDequeuedMessageId = MessageId.earliest;
 
                 clearIncomingMessages();
-                seekFuture.complete(null);
+                CompletableFuture<Void> future = null;
+                synchronized (this) {
+                    if (!hasParentConsumer && cnx() == null) {

Review Comment:
   I opened an issue: https://github.com/apache/pulsar/issues/22208
   
   It's not related to this PR but an existing test could be affected by this 
PR so I got it around.



-- 
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: commits-unsubscr...@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to