Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HIVE-27792 Corrected wrong hive-site.xml in docker #4796

Closed
wants to merge 1 commit into from

Conversation

xiaolin84250
Copy link
Contributor

hive.metastore.warehouse.dir, this configuration item is configured incorrectly, causing the configuration item to not take effect.

bug image:
image
image

after fixing:
image
image

What changes were proposed in this pull request?

I changed the packaging/src/docker/conf/hive-site.xml file.

Why are the changes needed?

I discovered this bug when using it. Since the default directory does not exist for non-root users in the container, it caused an error message without permission to create databases and data tables. After that, I corrected hive.metastore.warehouse.dir, this configuration item by rebuilding the image. This bug is solved.

The correct related configuration items are in the picture below.
image

Does this PR introduce any user-facing change?

No.

Is the change a dependency upgrade?

No.

How was this patch tested?

UT

hive.metastore.warehouse.dir, this configuration item is configured incorrectly, causing the configuration item to not take effect.
@xiaolin84250 xiaolin84250 changed the title HIVE-27787 Corrected wrong hive-site.xml HIVE-27787 Corrected wrong hive-site.xml in docker Oct 12, 2023
@sonarcloud
Copy link

sonarcloud bot commented Oct 12, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

warning The version of Java (11.0.8) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17.
Read more here

Copy link
Member

@ayushtkn ayushtkn left a comment

Choose a reason for hiding this comment

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

LGTM

@deniskuzZ
Copy link
Member

jira number looks unrelated to the change

@xiaolin84250 xiaolin84250 changed the title HIVE-27787 Corrected wrong hive-site.xml in docker HIVE-27792 Corrected wrong hive-site.xml in docker Oct 12, 2023
@xiaolin84250
Copy link
Contributor Author

jira 编号看起来与更改无关

Sorry, this is my first submission, I just created a new jira and got the number.
https://issues.apache.org/jira/browse/HIVE-27792

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants