LadyForest commented on a change in pull request #13011:
URL: https://github.com/apache/flink/pull/13011#discussion_r615554085



##########
File path: 
flink-table/flink-sql-parser-hive/src/main/codegen/includes/parserImpls.ftl
##########
@@ -252,6 +252,22 @@ SqlShowTables SqlShowTables() :
     }
 }
 
+/**
+* Parse a "Show Create Table" query command.
+*/
+SqlShowCreateTable  SqlShowCreateTable() :
+{
+    SqlIdentifier tableName;
+    SqlParserPos pos;
+}
+{
+    <SHOW> <CREATE> <TABLE> { pos = getPos();}

Review comment:
       > For Hive dialect, I don't think we should print the same result for 
SHOW CREATE TABLE.
   > 
   > Hive also provides a `SHOW CREATE TABLE` statement which should print the 
Hive DDL. So I would suggest we can postpone SHOW CREATE TABLE for hive to 
another issue.
   
   +1, I've reverted this change.




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


Reply via email to