Skip to content

[SPARK-48409][BUILD][TESTS] Upgrade MySQL & Postgres & Mariadb docker image version#46704

Closed
panbingkun wants to merge 3 commits intoapache:masterfrom
panbingkun:db_images_upgrade
Closed

[SPARK-48409][BUILD][TESTS] Upgrade MySQL & Postgres & Mariadb docker image version#46704
panbingkun wants to merge 3 commits intoapache:masterfrom
panbingkun:db_images_upgrade

Conversation

@panbingkun
Copy link
Copy Markdown
Contributor

@panbingkun panbingkun commented May 22, 2024

What changes were proposed in this pull request?

The pr aims to upgrade some db docker image version, include:

  • MySQL from 8.3.0 to 8.4.0
  • Postgres from 10.5.12 to 10.5.25
  • Mariadb from 16.2-alpine to 16.3-alpine

Why are the changes needed?

Tests dependencies upgrading.

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 SQL label May 22, 2024
@panbingkun panbingkun changed the title [Only Test] MySQL & Postgre Docker Images [SPARK-48409][BUILD][TESTS] Upgrade MySQL & Postgres & Mariadb docker image version May 24, 2024
@panbingkun panbingkun marked this pull request as ready for review May 24, 2024 07:27
override protected val keytabFileName = "postgres.keytab"

override val db = new DatabaseOnDocker {
override val imageName = sys.env.getOrElse("POSTGRES_DOCKER_IMAGE_NAME", "postgres:16.2")
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.

Let's align all postgres db docker images: 16.3-alpine

@panbingkun
Copy link
Copy Markdown
Contributor Author

cc @dongjoon-hyun @yaooqinn


override val db = new DatabaseOnDocker {
override val imageName = sys.env.getOrElse("MARIADB_DOCKER_IMAGE_NAME", "mariadb:10.5.12")
override val imageName = sys.env.getOrElse("MARIADB_DOCKER_IMAGE_NAME", "mariadb:10.5.25")
Copy link
Copy Markdown
Contributor Author

@panbingkun panbingkun May 24, 2024

Choose a reason for hiding this comment

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

I didn't see it yesterday, but it is today @yaooqinn 😄

@yaooqinn yaooqinn closed this in b15b6cf May 24, 2024
@yaooqinn
Copy link
Copy Markdown
Member

Merged to master. Thank you @panbingkun

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.

2 participants