tweise commented on a change in pull request #17111:
URL: https://github.com/apache/flink/pull/17111#discussion_r701428465
##########
File path:
flink-connectors/flink-connector-base/src/main/java/org/apache/flink/connector/base/source/hybrid/HybridSource.java
##########
@@ -92,13 +90,13 @@
private final List<SourceListEntry> sources;
// sources are populated per subtask at switch time
- private final Map<Integer, Source> switchedSources;
+ private final HybridSourceSplitSerializer.SwitchedSources switchedSources;
Review comment:
That came in after moving away from the fixed source sequence that
originally both, enumerator and serializer had access to. They still needed
access to the underlying serializer and therefore to the source that provided
that serializer. Now that serializers are decoupled, this hacky thing is no
longer needed. I just missed that in the refactor, thanks @stevenzwu for
catching it.
--
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]
