adamkotwasinski commented on a change in pull request #11843:
URL: https://github.com/apache/kafka/pull/11843#discussion_r820997952
##########
File path:
clients/src/main/java/org/apache/kafka/clients/FetchSessionHandler.java
##########
@@ -420,7 +420,7 @@ private String
topicIdPartitionsToLogString(Collection<TopicIdPartition> partiti
* @param topicPartitions The topicPartitions from the FetchResponse.
* @param ids The topic IDs from the FetchResponse.
* @param version The version of the FetchResponse.
- * @return True if the full fetch response partitions are
valid.
+ * @return null if the fetch response partitions are
valid; human-readable problem description otherwise.
Review comment:
I think we are fine - the implication comes from method signature
`verifyFullFetchResponsePartitions` and its Javadoc `Verify that a full fetch
response...`
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
