Skip to content

HIVE-27382: Migrate postgres-tpcds-metastore image to the Git repo#6490

Open
zabetak wants to merge 2 commits into
apache:masterfrom
zabetak:HIVE-27382-ci-publish
Open

HIVE-27382: Migrate postgres-tpcds-metastore image to the Git repo#6490
zabetak wants to merge 2 commits into
apache:masterfrom
zabetak:HIVE-27382-ci-publish

Conversation

@zabetak
Copy link
Copy Markdown
Member

@zabetak zabetak commented May 15, 2026

What changes were proposed in this pull request?

  1. Move necessary files from https://github.com/zabetak/hive-postgres-metastore repo to ASF.
  2. Remove redundant information and update README file.
  3. Create GitHub action for publishing new image to ASF Dockerhub on-demand.

Why are the changes needed?

  1. Facilitate maintenance
  2. Automate image publishing
  3. Track code provenance
  4. Avoid user-specific code silos

Does this PR introduce any user-facing change?

No

How was this patch tested?

The CI action publication was tested on my personal user namespace

I also pushed manually from my local env an image to the new apache/hive-postgres-tpcds-metastore namespace:
https://hub.docker.com/repository/docker/apache/hive-postgres-tpcds-metastore

Comment thread .github/workflows/hive-postgres-tpcds-metastore.yml Fixed
Copy link
Copy Markdown
Contributor

@abstractdog abstractdog left a comment

Choose a reason for hiding this comment

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

LGTM, pending tests
currently, there is an unrelated test failure :(

@zabetak
Copy link
Copy Markdown
Member Author

zabetak commented May 16, 2026

Thanks for the review @abstractdog ! The test failure is definitely unrelated so I will not relaunch the tests to save some resources.

zabetak added 2 commits May 27, 2026 16:51
1. Move necessary files from https://github.com/zabetak/hive-postgres-metastore repo to ASF.
2. Remove redundant information and update README file.
3. Create GitHub action for publishing new image to ASF Dockerhub on-demand.
@sonarqubecloud
Copy link
Copy Markdown

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