Skip to content

Conversation

@ayushtkn
Copy link
Member

@ayushtkn ayushtkn commented Feb 2, 2023

What changes were proposed in this pull request?

Allow creating table with only metadata location set.

Why are the changes needed?

Schema provided in the create command ain't used, everything depends on the metadata.json

Does this PR introduce any user-facing change?

Yes, Create command with metadata_location set doesn't require the schema

How was this patch tested?

UT

…inition when 'metadata_location' tblproperties is set.
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 2, 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 1 Code Smell

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

@ramesh0201
Copy link
Contributor

+1. LGTM

@ayushtkn ayushtkn merged commit 395a283 into apache:master Feb 7, 2023
yeahyung pushed a commit to yeahyung/hive that referenced this pull request Jul 20, 2023
…inition when 'metadata_location' tblproperties is set. (apache#4011). (Ayush Saxena, reviewed by Ramesh Kumar Thangarajan)
tarak271 pushed a commit to tarak271/hive-1 that referenced this pull request Dec 19, 2023
…inition when 'metadata_location' tblproperties is set. (apache#4011). (Ayush Saxena, reviewed by Ramesh Kumar Thangarajan)
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.

3 participants