rreddy-22 commented on code in PR #14147:
URL: https://github.com/apache/kafka/pull/14147#discussion_r1313240470
##########
group-coordinator/src/main/java/org/apache/kafka/coordinator/group/GroupCoordinatorShard.java:
##########
@@ -306,6 +308,22 @@ public CoordinatorResult<OffsetCommitResponseData, Record>
commitOffset(
return offsetMetadataManager.commitOffset(context, request);
}
+ /**
+ * Handles a OffsetCommit request.
+ *
+ * @param context The request context.
+ * @param request The actual OffsetCommit request.
+ *
+ * @return A Result containing the OffsetCommitResponse response and
Review Comment:
"A result containing the offset commit response", we don't want to use names
in the java doc, just the description right?
--
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
