Skip to content

[#12067] fix(jdbc-postgresql): complete V3 type compatibility#12083

Draft
nevzheng wants to merge 5 commits into
apache:mainfrom
nevzheng:codex/v3-phase2-postgresql
Draft

[#12067] fix(jdbc-postgresql): complete V3 type compatibility#12083
nevzheng wants to merge 5 commits into
apache:mainfrom
nevzheng:codex/v3-phase2-postgresql

Conversation

@nevzheng

Copy link
Copy Markdown
Collaborator

What changes were proposed in this pull request?

Define and document PostgreSQL compatibility for the five Gravitino V3-native type families.

Timestamp precision greater than 6, Variant, Unknown/Null, Geometry, and Geography writes are rejected before SQL. Native JSON and PostGIS reads remain explicit external types until their complete metadata can be recovered faithfully.

Why are the changes needed?

JSON/JSONB do not implement the complete Variant contract, and the current PostGIS read path cannot preserve all subtype, SRID, and Geography edge-algorithm semantics. Early rejection prevents a lossy write contract.

Fix: #12067

Related: #12056

Shared JDBC preflight: #12065

Shared JDBC preflight PR: #12081

Does this PR introduce any user-facing change?

Yes. Unsupported V3-native writes now fail before SQL generation instead of relying on an incomplete database mapping.

How was this patch tested?

  • Added converter and SQL-generation rejection coverage.
  • Covered Docker create rejection and no-table behavior.
  • Kept native JSON and PostGIS reads explicit.
  • Updated docs/jdbc-postgresql-catalog.md.

@github-actions

Copy link
Copy Markdown

Code Coverage Report

Overall Project 67.89% +0.04% 🟢
Files changed 99.02% 🟢

Module Coverage
aliyun 1.72% 🔴
api 47.82% 🟢
authorization-common 85.96% 🟢
aws 42.04% 🟢
azure 2.47% 🔴
catalog-common 9.92% 🔴
catalog-fileset 80.23% 🟢
catalog-glue 66.91% 🟢
catalog-hive 79.42% 🟢
catalog-jdbc-clickhouse 81.74% 🟢
catalog-jdbc-common 44.22% 🟢
catalog-jdbc-doris 81.6% 🟢
catalog-jdbc-hologres 54.03% 🟢
catalog-jdbc-mysql 79.23% 🟢
catalog-jdbc-oceanbase 80.91% 🟢
catalog-jdbc-postgresql 82.76% +3.27% 🟢
catalog-jdbc-starrocks 78.51% 🟢
catalog-kafka 77.01% 🟢
catalog-lakehouse-generic 59.18% 🟢
catalog-lakehouse-hudi 79.1% 🟢
catalog-lakehouse-iceberg 86.56% 🟢
catalog-lakehouse-paimon 84.25% 🟢
catalog-model 77.72% 🟢
cli 44.5% 🟢
client-java 78.29% 🟢
common 51.95% 🟢
core 82.53% 🟢
filesystem-hadoop3 77.3% 🟢
flink 0.0% 🔴
flink-common 47.09% 🟢
flink-runtime 0.0% 🔴
gcp 14.12% 🔴
hadoop-auth 68.0% 🟢
hadoop-common 12.7% 🔴
hive-metastore-common 53.29% 🟢
iceberg-common 63.63% 🟢
iceberg-rest-server 74.9% 🟢
idp-basic 86.02% 🟢
integration-test-common 0.0% 🔴
jobs 66.17% 🟢
lance-common 20.67% 🔴
lance-rest-server 64.84% 🟢
lineage 53.02% 🟢
optimizer 83.17% 🟢
optimizer-api 21.95% 🔴
server 85.97% 🟢
server-common 75.78% 🟢
spark 28.57% 🔴
spark-common 46.01% 🟢
tencent 69.84% 🟢
trino-connector 40.29% 🟢
Files
Module File Coverage
catalog-jdbc-postgresql PostgreSqlTypeConverter.java 99.02% 🟢

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.

[Subtask] Complete V3 type compatibility for the PostgreSQL catalog

1 participant