close
Skip to content

[Feature][Connector-V2] Postgresql support read TIMESTAMP_TZ#10048

Merged
Carl-Zhou-CN merged 19 commits into
apache:devfrom
yzeng1618:Feature-support-jdbc-postgresql
Nov 29, 2025
Merged

[Feature][Connector-V2] Postgresql support read TIMESTAMP_TZ#10048
Carl-Zhou-CN merged 19 commits into
apache:devfrom
yzeng1618:Feature-support-jdbc-postgresql

Conversation

@yzeng1618

Copy link
Copy Markdown
Member

#9785

Purpose of this pull request

Add support for reading TIMESTAMP_TZ type in Postgresql

Does this PR introduce any user-facing change?

No breaking change; stronger TIMESTAMP_TZ read/write consistency with graceful fallbacks for legacy drivers

How was this patch tested?

Unit tests: adjust type converter tests for Postgresql to expect TIMESTAMP_TZ
E2E: Postgresql TIMESTAMP_TZ (part-3) added/fixed; compare offset-included strings for losslessness
Build: modular build and local install; spotless formatting passed

Check list

@yzeng1618 yzeng1618 requested a review from davidzollo November 13, 2025 06:25

@yzeng1618 yzeng1618 left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

All revisions have been completed.

Carl-Zhou-CN
Carl-Zhou-CN previously approved these changes Nov 27, 2025

@Carl-Zhou-CN Carl-Zhou-CN left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@yzeng1618 yzeng1618 requested a review from davidzollo November 28, 2025 01:08

@Carl-Zhou-CN Carl-Zhou-CN left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

+1

@Carl-Zhou-CN Carl-Zhou-CN merged commit 30b0a08 into apache:dev Nov 29, 2025
6 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.

4 participants