Skip to content

Draft: FLINK-28229 (CI)#23557

Closed
afedulov wants to merge 5 commits intoapache:masterfrom
afedulov:28229-collection
Closed

Draft: FLINK-28229 (CI)#23557
afedulov wants to merge 5 commits intoapache:masterfrom
afedulov:28229-collection

Conversation

@afedulov
Copy link
Copy Markdown
Contributor

@afedulov afedulov commented Oct 19, 2023

No description provided.

…ate module

This is required to avoid circular dependency that arises when DataGeneratorSource gets used in flink-streaming-java
…treaming-java

Reason for this change: OutputTypeConfigurable needs to be used in DataGeneratorSource (see StreamExecutionEnvironmentTest#testFromElementsPostConstructionType). OutputTypeConfigurable is located in StreamExecutionEnvironment. When DataGeneratorSource gets added into flink-streaming-java, this creates a cycle. Marker interfaces should ideally reside in flink-core (InputTypeConfigurable already does).
- Deprecates OutputTypeConfigurable in flink-streaming-java
- Adds a new marker OutputTypeConfigurable interface to flink-core
- Modifies operators that use this interface to support both versions
@afedulov afedulov changed the title Draft: FLINK-28050 (CI) Draft: FLINK-28229 (CI) Oct 19, 2023
@afedulov afedulov marked this pull request as ready for review October 19, 2023 22:58
@afedulov
Copy link
Copy Markdown
Contributor Author

@flinkbot run azure

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants