Re: [PR] [FLINK-39792] Add multi-headers metadata key to Kafka connector [flink-connector-kafka]

["AHeise (via GitHub)" <gi...@apache.org>]

AHeise commented on code in PR #258:
URL: https://github.com/apache/flink-connector-kafka/pull/258#discussion_r3341419199


##########
flink-connector-kafka/src/main/java/org/apache/flink/streaming/connectors/kafka/table/KafkaDynamicSource.java:
##########
@@ -623,6 +625,42 @@ public Object read(ConsumerRecord<?, ?> record) {
                     }
                 }),
 
+        /**
+         * Wire-faithful alternative to {@code headers}: preserves duplicate keys and insertion
+         * order using {@code ARRAY<ROW<key STRING, value BYTES>>} instead of a lossy MAP.
+         */
+        MULTI_HEADERS(
+                "multi-headers",

Review Comment:
   @loserwang1024 WDYT about the ideas tossed around? Any preference?



-- 
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: issues-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org