merlimat commented on a change in pull request #8992:
URL: https://github.com/apache/pulsar/pull/8992#discussion_r546073784



##########
File path: 
pulsar-broker/src/main/java/org/apache/pulsar/broker/service/ServerCnx.java
##########
@@ -1192,6 +1193,17 @@ protected void handleProducer(final CommandProducer 
cmdProducer) {
                                     }
                                     return null;
                                 });
+
+                                producerQueuedFuture.thenRun(() -> {
+                                    // If the producer is queued waiting, we 
will get an immediate notification
+                                    // that we need to pass to client
+                                    if (isActive()) {
+                                        log.info("[{}] Producer is waiting in 
qeuue: {}", remoteAddress, producer);

Review comment:
       ```suggestion
                                           log.info("[{}] Producer is waiting 
in queue: {}", remoteAddress, producer);
   ```




----------------------------------------------------------------
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]


Reply via email to