kasakrisz opened a new pull request, #3802: URL: https://github.com/apache/hive/pull/3802
### What changes were proposed in this pull request? Refactor the way ctas is executed: * Do not create the table in `HiveIcebergSerDe` since it is also created at compile time. * Add a DDLTask before the `TezTask` to create the Iceberg table. * Collect the properties added to jobconfig from the Serde object and location and fileio from HiveCatalog. Location can be calculated at compile time using `SemanticAnalyzer.getCtasOrCMVLocation` * Persist the new table metaobject to a temp file when committing the table creation. * Read back the table metaobject anytime it is required from the TezTask and the MoveTask. ### Why are the changes needed? See jira. ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? ``` mvn test -Dtest.output.overwrite -Dtest=TestIcebergCliDriver -Dqfile=ctas_iceberg_partitioned_orc.q -pl itests/qtest-iceberg -Piceberg -Pitests -Drat.skip ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
