imay commented on a change in pull request #3819:
URL: https://github.com/apache/incubator-doris/pull/3819#discussion_r439286886
##########
File path: fe/src/main/cup/sql_parser.cup
##########
@@ -1244,6 +1244,15 @@ data_desc ::=
RESULT = new DataDescription(tableName, partitionNames, files,
colList, colSep, fileFormat,
columnsFromPath, isNeg, colMappingList, whereExpr);
:}
+ | KW_DATA KW_FROM KW_TABLE ident:srcTableName
+ opt_negative:isNeg
+ KW_INTO KW_TABLE ident:tableName
+ opt_partition_names:partitionNames
+ opt_col_mapping_list:colMappingList
Review comment:
@morningman
What you suggested seems more flexible. however I think it is a more complex
feature.
In my opinion, we can only support table directlly to fulfill 80% user
requirements.
And we can implement what your suggest later when it is needed.
And we can keep the same with load file in column mapping by column_lists
and SET clause.
----------------------------------------------------------------
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]
