Skip to content

[Cherry-pick to branch-1.3] [#11795] fix(docker): Upgrade bundled PostgreSQL JDBC driver (#11796) - #12697

Merged
jerryshao merged 1 commit into
apache:branch-1.3from
diqiu50:cherry-pick-11796-to-branch-1.3
Aug 28, 2026
Merged

[Cherry-pick to branch-1.3] [#11795] fix(docker): Upgrade bundled PostgreSQL JDBC driver (#11796)#12697
jerryshao merged 1 commit into
apache:branch-1.3from
diqiu50:cherry-pick-11796-to-branch-1.3

Conversation

@diqiu50

@diqiu50 diqiu50 commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Cherry-pick Information:

  • Original commit: 95c07c7
  • Target branch: branch-1.3
  • Status: ⚠️ Cherry-pick with conflict resolution — kept branch-1.3's rewritten docs/how-to-use-relational-backend-storage.md wording (already recommends latest driver version) instead of the incoming pinned-version text; all code/script changes applied cleanly.

…pache#11796)

Upgrade the PostgreSQL JDBC driver downloaded into Docker and test
helper packages from `42.7.0` to `42.7.11`.

This updates the Gravitino Docker dependency script, Trino Docker/test
helper references, integration test download URL, and documentation
link.

The Gravitino 1.3.0-rc6 image scan reports CRITICAL `CVE-2024-1597` for
`/opt/gravitino/jdbc-drivers/postgresql-42.7.0.jar`.

Fix: apache#11795

No API or configuration property changes. Docker/test helper scripts and
documentation now reference PostgreSQL JDBC `42.7.11`.

- Confirmed no `postgresql-42.7.0` references remain in the touched
areas.
- Verified `https://jdbc.postgresql.org/download/postgresql-42.7.11.jar`
returns HTTP 200.

(cherry picked from commit 95c07c7)
Copilot AI lite review requested due to automatic review settings August 28, 2026 02:05

Copilot AI 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.

Pull request overview

This cherry-pick updates the bundled PostgreSQL JDBC driver references on branch-1.3 to the newer 42.7.11 release, ensuring the Docker images and integration tooling download/copy the upgraded driver consistently.

Changes:

  • Bump PostgreSQL JDBC driver from 42.7.0 to 42.7.11 across Docker packaging scripts, Dockerfile, and integration tooling.
  • Fix the Trino dependency script’s PostgreSQL driver existence check to correctly expand ${PG_VERSION}.
  • Update integration-test shared constant to download the new driver version.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
trino-connector/integration-test/trino-test-tools/download_jar.sh Update PostgreSQL driver JAR name/URL and copied artifact for IT tooling.
integration-test-common/src/test/java/org/apache/gravitino/integration/test/util/BaseIT.java Update PostgreSQL JDBC driver download URL constant used by IT setup.
dev/docker/trino/trino-dependency.sh Bump PG_VERSION and fix the file-existence check to use ${PG_VERSION} correctly.
dev/docker/trino/Dockerfile Copy the upgraded PostgreSQL JDBC driver into the Trino iceberg plugin directory.
dev/docker/gravitino/gravitino-dependency.sh Download the upgraded PostgreSQL JDBC driver into the Gravitino docker package.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment on lines +57 to 61
download_jar "postgresql-42.7.11.jar" \
"https://jdbc.postgresql.org/download/postgresql-42.7.11.jar" \
"$GRAVITINO_SERVER_DIR/catalogs/jdbc-postgresql/libs"
cp -rp $GRAVITINO_SERVER_DIR/catalogs/jdbc-postgresql/libs/postgresql-42.7.0.jar $GRAVITINO_SERVER_DIR/catalogs/lakehouse-iceberg/libs
cp -rp $GRAVITINO_SERVER_DIR/catalogs/jdbc-postgresql/libs/postgresql-42.7.11.jar $GRAVITINO_SERVER_DIR/catalogs/lakehouse-iceberg/libs
}
@github-actions

Copy link
Copy Markdown

Code Coverage Report

Overall Project 67.65% -0.78% 🟢
Files changed 26.52% 🔴

Module Coverage
aliyun 1.72% 🔴
api 47.73% 🟢
authorization-common 85.96% 🟢
aws 42.04% 🟢
azure 2.47% 🔴
catalog-common 10.4% 🔴
catalog-fileset 80.23% 🟢
catalog-glue 68.95% 🟢
catalog-hive 79.42% 🟢
catalog-jdbc-clickhouse 81.67% 🟢
catalog-jdbc-common 45.48% 🟢
catalog-jdbc-doris 82.36% 🟢
catalog-jdbc-hologres 54.03% 🟢
catalog-jdbc-mysql 79.33% 🟢
catalog-jdbc-oceanbase 78.6% 🟢
catalog-jdbc-postgresql 83.39% 🟢
catalog-jdbc-starrocks 79.16% 🟢
catalog-kafka 77.01% 🟢
catalog-lakehouse-generic 59.18% 🟢
catalog-lakehouse-hudi 79.1% 🟢
catalog-lakehouse-iceberg 85.86% 🟢
catalog-lakehouse-paimon 82.14% 🟢
catalog-model 77.72% 🟢
cli 44.51% 🟢
client-java 77.99% 🟢
common 50.29% -0.15% 🟢
core 82.68% 🟢
filesystem-hadoop3 77.3% 🟢
flink 0.0% 🔴
flink-common 47.12% 🟢
flink-runtime 0.0% 🔴
gcp 14.12% 🔴
hadoop-common 10.88% 🔴
hive-metastore-common 53.77% 🟢
iceberg-aliyun-bundle 0.0% 🔴
iceberg-common 58.15% 🟢
iceberg-rest-server 73.9% 🟢
idp-basic 86.02% 🟢
integration-test-common 0.0% 🔴
jobs 66.17% 🟢
lance-common 24.42% 🔴
lance-rest-server 60.13% 🟢
lineage 53.02% 🟢
optimizer 82.95% 🟢
optimizer-api 21.95% 🔴
server 86.46% 🟢
server-common 76.03% 🟢
spark 28.57% 🔴
spark-common 41.66% 🟢
trino-connector 40.25% -10.05% 🟢
Files
Module File Coverage
common PartitionStatisticsUpdateDTO.java 0.0% 🔴
trino-connector CatalogPropertyConverter.java 100.0% 🟢
GravitinoAuthProvider.java 81.2% 🟢
DropCatalogStoredProcedure.java 53.85% 🔴
GravitinoConnectorPluginManager.java 41.22% 🔴
MySQLMetadataAdapter.java 37.23% 🔴
CatalogConnectorMetadataAdapter.java 33.33% 🔴
CatalogConnectorManager.java 29.78% 🔴
GravitinoConnector.java 28.44% 🔴
GravitinoMetadata.java 15.95% 🔴
CatalogConnectorMetadata.java 15.67% 🔴
CatalogRegister.java 0.89% 🔴
GravitinoConnectorFactory.java 0.0% 🔴
DefaultCatalogConnectorFactory.java 0.0% 🔴
AlterCatalogStoredProcedure.java 0.0% 🔴
CreateCatalogStoredProcedure.java 0.0% 🔴

@jerryshao
jerryshao merged commit 8b2dfbb into apache:branch-1.3 Aug 28, 2026
35 checks passed
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.

4 participants