rreddy-22 commented on code in PR #14467: URL: https://github.com/apache/kafka/pull/14467#discussion_r1353043290
########## group-coordinator/src/main/java/org/apache/kafka/coordinator/group/Group.java: ########## @@ -117,4 +119,16 @@ void validateOffsetFetch( * @param records The list of records. */ void createGroupTombstoneRecords(List<Record> records); + + /** + * @return Whether the group can be deleted or not. + */ + boolean isEmpty(); Review Comment: the return value seems more like a use case right? Should we update the name of the method or the return statement and add the "whether group can be deleted or not" part as a use case in the javadoc -- 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