HeartSaVioR edited a comment on issue #27107: [SPARK-30436][SQL] Allow CREATE 
EXTERNAL TABLE with only requiring LOCATION
URL: https://github.com/apache/spark/pull/27107#issuecomment-571452747
 
 
   The parts differentiate between two syntaxes are skewSpec, rowFormat, and 
createFileFormat (using any of them would make create statement go into 2nd 
syntax), and all of them are optional. We're not enforcing to specify it but 
rely on the parser.
   
   Why don't we change the syntax to require some of them for createHiveTable 
(I'm not sure which should be required... STORED AS? anyway at least we can 
differentiate explicitly to require any of these three using `+`), and document 
two different syntaxes for create table and explain the difference? (Looks like 
the doc page for create table has no content, assuming there's a plan to 
address it.)
   
   And then I don't see a reason to retain EXTERNAL to first syntax as it will 
just make statement fail.

----------------------------------------------------------------
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]

Reply via email to