Skip to content

[SPARK-46060][BUILD][TESTS] Upgrade MySQL/MariaDB/PostgreSQL/DB2 test dependencies#43963

Closed
panbingkun wants to merge 1 commit intoapache:masterfrom
panbingkun:upgrade_jdbc_driver
Closed

[SPARK-46060][BUILD][TESTS] Upgrade MySQL/MariaDB/PostgreSQL/DB2 test dependencies#43963
panbingkun wants to merge 1 commit intoapache:masterfrom
panbingkun:upgrade_jdbc_driver

Conversation

@panbingkun
Copy link
Contributor

@panbingkun panbingkun commented Nov 22, 2023

What changes were proposed in this pull request?

The pr aims to upgrade jdbc related test dependencies, include:

  • com.mysql:mysql-connector-j from 8.0.33 to 8.2.0
  • org.mariadb.jdbc:mariadb-java-client from 2.7.9 to 2.7.11
  • org.postgresql:postgresql from 42.6.0 to 42.7.0
  • com.ibm.db2:jcc from 11.5.8.0 to 11.5.9.0

Why are the changes needed?

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Pass GA.

Was this patch authored or co-authored using generative AI tooling?

No.

@github-actions github-actions bot added the BUILD label Nov 22, 2023
@panbingkun panbingkun changed the title Upgrade jdbc driver [SPARK-46060][BUILD][TESTS] Upgrade jdbc related test dependencies Nov 23, 2023
@panbingkun panbingkun marked this pull request as ready for review November 23, 2023 02:53
@LuciferYang
Copy link
Contributor

LuciferYang commented Nov 23, 2023

Can't com.microsoft.sqlserver:mssql-jdbc be upgraded? and org.apache.derby:derby?

@dongjoon-hyun dongjoon-hyun changed the title [SPARK-46060][BUILD][TESTS] Upgrade jdbc related test dependencies [SPARK-46060][BUILD][TESTS] Upgrade MySQL/MariaDB/PostgreSQL/DB2 test dependencies Nov 29, 2023
Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

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

+1, LGTM.

To @panbingkun and @LuciferYang , I revised the PR title to fit with the content of the PR.

For MSSQL Server and derby, we can handle later when we have more time.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants