Skip to content

HIVE-27798: Corrected wrong hive-site.xml - #4803

Merged
ayushtkn merged 1 commit into
apache:masterfrom
xiaolin84250:HIVE-27798
Oct 16, 2023
Merged

HIVE-27798: Corrected wrong hive-site.xml#4803
ayushtkn merged 1 commit into
apache:masterfrom
xiaolin84250:HIVE-27798

Conversation

@xiaolin84250

Copy link
Copy Markdown
Contributor

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

bug image:
image
image

fix image:
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 I was using it. Since there is no default directory for non-root users in the container, error messages appear when creating databases and data tables without permission. I corrected hive.metastore.warehouse.dir configuration. This error has been resolved.

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

Does this PR introduce any user-facing change?

Now there will be no errors when creating databases and tables in docker containers, and the data and directories will appear in the specified directory.

Is the change a dependency upgrade?

No

How was this patch tested?

UT

Correct the configuration item: hive.metastore.warehouse.dir
@xiaolin84250

xiaolin84250 commented Oct 14, 2023

Copy link
Copy Markdown
ContributorAuthor

Sorry, this is my first time submitting Pull Requests. #4796 jira number was wrong, so I resubmit.
This is my issue jira: https://issues.apache.org/jira/browse/HIVE-27798

@zhangbutao

Copy link
Copy Markdown
Contributor

Please don't create duplicate PR,one is enough.

@ayushtknayushtkn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@sonarqubecloud

Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed! Quality Gate passed

BugA0 Bugs
VulnerabilityA0 Vulnerabilities
Security HotspotA0 Security Hotspots
Code SmellA0 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

@ayushtkn
ayushtkn merged commit 357714a into apache:masterOct 16, 2023
tarak271 pushed a commit to tarak271/hive-1 that referenced this pull request Dec 19, 2023
Sign up for freeto 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.

4 participants

@xiaolin84250@zhangbutao@ayushtkn@asf-ci-hive