philipnee commented on code in PR #13490:
URL: https://github.com/apache/kafka/pull/13490#discussion_r1192767321
##########
clients/src/main/java/org/apache/kafka/clients/consumer/internals/PrototypeAsyncConsumer.java:
##########
@@ -546,6 +619,22 @@ public ConsumerRecords<K, V> poll(long timeout) {
throw new KafkaException("method not implemented");
}
+ private void maybeWakeup() {
+ if (this.closed)
+ throw new IllegalStateException("This consumer has already been
closed.");
+
+ log.debug("Raising WakeupException in response to user wakeup");
Review Comment:
This is actually in the original wakeup call see: (I forgot why I left it
out of the getAndSet block, but I'll move it into the block.
```
if (!wakeupDisabled.get() && wakeup.get()) {
log.debug("Raising WakeupException in response to user wakeup");
wakeup.set(false);
throw new WakeupException();
}
```
--
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: jira-unsubscr...@kafka.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
