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


##########
flink-connector-kafka/src/main/java/org/apache/flink/connector/kafka/sink/KafkaWriterStateSerializer.java:
##########
@@ -59,14 +61,20 @@ public KafkaWriterState deserialize(int version, byte[] 
serialized) throws IOExc
         try (final ByteArrayInputStream bais = new 
ByteArrayInputStream(serialized);
                 final DataInputStream in = new DataInputStream(bais)) {
             final String transactionalIdPrefix = in.readUTF();
+            int subtaskId = 0;

Review Comment:
   It's a good point. I suggest to add a special variable UNKNOWN=-1 and 
disallow using POOLING then. In that way, we can guide the user to safely 
convert the checkpoint first to INCREMENTING and then POOLING.



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