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-26628: Iceberg table is created when running explain ctas command #3745
HIVE-26628: Iceberg table is created when running explain ctas command #3745
Conversation
ed7ce0f
to
7843b53
Compare
732850f
to
4cc5f8f
Compare
e471d72
to
7b91cb9
Compare
add q test
f328f1c
to
33a1ec6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM +1, pending tests
Kudos, SonarCloud Quality Gate passed! |
Table Parameters: | ||
COLUMN_STATS_ACCURATE {\"BASIC_STATS\":\"true\",\"COLUMN_STATS\":{\"a\":\"true\",\"b\":\"true\",\"c\":\"true\"}} | ||
EXTERNAL TRUE | ||
bucketing_version -1 | ||
engine.hive.enabled true | ||
iceberg.orc.files.only true | ||
metadata_location hdfs://### HDFS PATH ### | ||
numFiles 2 | ||
numRows 2 | ||
previous_metadata_location hdfs://### HDFS PATH ### | ||
serialization.format 1 | ||
storage_handler org.apache.iceberg.mr.hive.HiveIcebergStorageHandler | ||
table_type ICEBERG | ||
totalSize 812 | ||
#### A masked pattern was here #### | ||
uuid #Masked# | ||
write.format.default orc | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kasakrisz / @deniskuzZ The table created was a v2 table but here the property is missing & we need to set merge-on-read also for any v2 table, those properties also seems missing. Can you folks check
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ayushtkn
Thanks for the notice.
I try to fix this in a follow-up patch focusing on the way ctas plan is generated and the statement is executed.
https://issues.apache.org/jira/browse/HIVE-26771
What changes were proposed in this pull request?
Why are the changes needed?
Explain command should not make changes in the catalog.
Does this PR introduce any user-facing change?
No.
How was this patch tested?