Skip to content

[FLINK-39760] Fix missing JTS core dependency of PostgreSQL YAML connector#4467

Merged
lvyanquan merged 2 commits into
apache:masterfrom
haruki-830:FLINK-39760
Jul 10, 2026
Merged

[FLINK-39760] Fix missing JTS core dependency of PostgreSQL YAML connector#4467
lvyanquan merged 2 commits into
apache:masterfrom
haruki-830:FLINK-39760

Conversation

@haruki-830

Copy link
Copy Markdown
Contributor

Summary

This commit adds the missing JTS core dependency to the PostgreSQL YAML connector, ensuring the connector can be built independently when geometry handling code uses JTS classes.

Key Changes

Explicit JTS Core Dependency

  • Added org.locationtech.jts:jts-core to flink-cdc-pipeline-connector-postgres.
  • Uses jts.version set to 1.19.0, matching the JTS version already used transitively in the project dependency tree.

Fix Standalone Connector Build

  • PostgresEventDeserializer imports and uses JTS classes such as Coordinate and WKBReader for geometry value handling.
  • Before this change, the PostgreSQL YAML connector did not declare jts-core in its own module dependencies, which could break isolated module builds or packaging workflows.

JIRA Reference

https://issues.apache.org/jira/browse/FLINK-39760

@haruki-830 haruki-830 marked this pull request as ready for review July 9, 2026 11:25
<include>com.fasterxml.*:*</include>
<!-- Include fixed version 30.1.1-jre-14.0 of flink shaded guava -->
<include>org.apache.flink:flink-shaded-guava</include>
</includes>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please also add org.locationtech.jts:jts-core to the shade includes

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Addressed, thanks.

@lvyanquan lvyanquan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

+1.

@lvyanquan lvyanquan merged commit e780a7c into apache:master Jul 10, 2026
36 of 39 checks passed
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