ijuma commented on a change in pull request #8705:
URL: https://github.com/apache/kafka/pull/8705#discussion_r431583127
##########
File path: clients/src/main/java/org/apache/kafka/common/network/Selector.java
##########
@@ -804,7 +804,33 @@ private void maybeCloseOldestConnection(long
currentTimeNanos) {
}
/**
- * Clear the results from the prior poll
+ * Clears completed receives. This is used by SocketServer to remove
references to
+ * receive buffers after processing completed receives, without waiting
for the next
+ * poll() after all results have been processed.
Review comment:
Nit: maybe `after all results have been processed` is a bit redundant?
Same for the `clearCompletedSends` docs.
----------------------------------------------------------------
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:
us...@infra.apache.org
