OmniaGM commented on code in PR #15968:
URL: https://github.com/apache/kafka/pull/15968#discussion_r1884163619


##########
core/src/main/scala/kafka/server/KafkaApis.scala:
##########
@@ -2478,7 +2493,8 @@ class KafkaApis(val requestChannel: RequestChannel,
           } else {
             // Otherwise, the regular appendRecords path is used for all the 
non __consumer_offsets
             // partitions or for all partitions when the new group coordinator 
is disabled.
-            controlRecords += partition -> 
MemoryRecords.withEndTransactionMarker(
+            // If topicIdPartition contains Uuid.ZERO_UUid or null all 
functionality will fall back on topic name.

Review Comment:
   you are right I have updated this 



-- 
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

Reply via email to