cloud-fan commented on issue #28026: [SPARK-31257][SQL] Unify create table syntax URL: https://github.com/apache/spark/pull/28026#issuecomment-611472963 Supporting the Hive style PARTITION BY is very straightforward and I don't see other opinions. Supporting the Hive style CREATE TABLE completely in v2 catalog API does have some different opinions: 1. shall we support the EXTERNAL keyword? As I said in https://github.com/apache/spark/pull/28026#issuecomment-606746750, it's not useful IMO. 2. what should be the option key names for these Hive specific information? I'm not saying that this PR makes the wrong decision, but including more work other than "unifying the CREATE TABLE syntax" does trigger more discussions and makes it harder to get this PR merged.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
