Skip to content

[flink] Implement FLIP-314 LineageVertexProvider for source and sink connectors#7311

Open
jsingh-yelp wants to merge 1 commit intoapache:masterfrom
jsingh-yelp:add-flip-314-flink-lineage-support
Open

[flink] Implement FLIP-314 LineageVertexProvider for source and sink connectors#7311
jsingh-yelp wants to merge 1 commit intoapache:masterfrom
jsingh-yelp:add-flip-314-flink-lineage-support

Conversation

@jsingh-yelp
Copy link
Contributor

@jsingh-yelp jsingh-yelp commented Feb 27, 2026

Purpose

Tests

API and Format

  • This change does not change any API
  • Note: This PR changes how PaimonDataStreamScanProvider and PaimonDataStreamSinkProvider are constructed callers now use createProvider() factory methods instead of direct constructors. This is necessary because the FLIP-314 implementation requires the returned ScanRuntimeProvider/SinkRuntimeProvider to also implement LineageVertexProvider, an interface that only exists in Flink 1.20+. To maintain backward compatibility with older Flink versions, the factory methods delegate to a version-specific DataStreamProviderFactory: the Flink 2.x variant wraps the provider with lineage support, while the Flink 1.x stub returns it unchanged.

Documentation

  • Yes I think this is worth adding documentation for, I am happy to add docs for it if someone can point me where to add the specific documentation for this.
  • Yes this introduces a new feature of adding support of native lineage generation of Paimon tables for Flink.

Note: This change also depends on two other changes and one of them is already merged. Other two changes:

@jsingh-yelp jsingh-yelp force-pushed the add-flip-314-flink-lineage-support branch 5 times, most recently from 7fc3948 to de23686 Compare March 4, 2026 15:06
@jsingh-yelp jsingh-yelp force-pushed the add-flip-314-flink-lineage-support branch from de23686 to c1e883e Compare March 4, 2026 16:04
@jsingh-yelp jsingh-yelp marked this pull request as ready for review March 4, 2026 18:21
</dependency>
<dependency>
<groupId>org.apache.flink</groupId>
<artifactId>flink-table-api-java-bridge</artifactId>
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jsingh-yelp
Copy link
Contributor Author

@JingsongLi can I please have a review on this

@JingsongLi
Copy link
Contributor

Cool, @yunfengzhou-hub can you take a look?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants