lucliu1108 commented on code in PR #22639:
URL: https://github.com/apache/kafka/pull/22639#discussion_r3462088632


##########
streams/src/main/java/org/apache/kafka/streams/processor/internals/StreamThread.java:
##########
@@ -692,13 +693,24 @@ private static StreamsRebalanceData 
initStreamsRebalanceData(final UUID processI
 
         final Map<String, StreamsRebalanceData.Subtopology> subtopologies = 
initBrokerTopology(config, internalTopologyBuilder);
 
-        return new StreamsRebalanceData(
+        final StreamsRebalanceData streamsRebalanceData = new 
StreamsRebalanceData(
             processId,
             endpoint,
             rackId,
             subtopologies,
             config.getClientTags()
         );
+
+        if 
(config.getBoolean(StreamsConfig.TOPOLOGY_DESCRIPTION_PUSH_ENABLED_CONFIG)) {
+            final TopologyDescription description = 
internalTopologyBuilder.describe();

Review Comment:
   Fix: Modified the wire converter to accept an additional topicName 
decorator, in this way we could convert internal topic names produced by 
`internalTopologyBuilder.describe()` to the format with `app-id`.



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

Reply via email to