gatorsmile commented on pull request #28026: URL: https://github.com/apache/spark/pull/28026#issuecomment-682018978
@rdblue, unifying CREATE TABLE syntax is important for Spark. We need to finish this PR. Currently, I saw the major argument is about the EXTERNAL table property. Can we move on and split this PR into smaller ones to unblock the work? 1. The main PR that unifies the syntax, and puts serde info in the statement plan. We should fail to convert statement to v2 create plan if serde info is present. 2. Support converting statement to v2 create plan when serde info is present. 3. Add a `option` namespace to distinguish SERDEPROPERTIES and OPTIONS in the table properties 4. Add EXTERNAL table property. I think the baseline is to avoid any breaking change. If we want to introduce a breaking change, we need to discuss it one by one. cc @HeartSaVioR @cloud-fan ---------------------------------------------------------------- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
