tweise commented on a change in pull request #17111:
URL: https://github.com/apache/flink/pull/17111#discussion_r702094543
##########
File path:
flink-connectors/flink-connector-base/src/main/java/org/apache/flink/connector/base/source/hybrid/HybridSourceSplit.java
##########
@@ -19,33 +19,45 @@
package org.apache.flink.connector.base.source.hybrid;
import org.apache.flink.api.connector.source.SourceSplit;
+import org.apache.flink.core.io.SimpleVersionedSerializer;
+import java.io.IOException;
import java.util.ArrayList;
+import java.util.Arrays;
import java.util.List;
import java.util.Objects;
/** Source split that wraps the actual split type. */
public class HybridSourceSplit implements SourceSplit {
- private final SourceSplit wrappedSplit;
+ private final byte[] wrappedSplitBytes;
+ private final int wrappedSplitSerializerVersion;
Review comment:
Considered that as well and prefer the verbose name to make clear that
this is the serializer version for the underlying state vs that of
`HybridSourceSplit`.
--
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-unsubscr...@flink.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
