zhilinli123 commented on code in PR #5992:
URL: https://github.com/apache/seatunnel/pull/5992#discussion_r1537496237
##########
seatunnel-api/src/main/java/org/apache/seatunnel/api/serialization/DeserializationSchema.java:
##########
@@ -34,12 +35,30 @@ public interface DeserializationSchema<T> extends
Serializable {
*/
T deserialize(byte[] message) throws IOException;
+ /**
+ * Deserializes the byte message.
+ *
+ * @param message The message, as a byte array.
+ * @param tablePath The Registry name.
+ * @return The deserialized message as an SeaTunnel Row (null if the
message cannot be
+ * deserialized).
+ */
+ T deserialize(byte[] message, TablePath tablePath) throws IOException;
Review Comment:
> 1. A topic should not have multiple data formats
> 2. Each topic should correspond to a fixed data format
The current logic should only have one data format per topic, if it is a
regular matching topic will also force a format, unless it can be different
topics we can let the user set different formats
--
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]