xiacongling commented on code in PR #28083:
URL: https://github.com/apache/doris/pull/28083#discussion_r1699270179
##########
fe/fe-core/src/main/antlr4/org/apache/doris/nereids/DorisParser.g4:
##########
@@ -62,9 +62,9 @@ statement
SET updateAssignmentSeq
fromClause?
whereClause #update
- | explain? cte? DELETE FROM tableName=multipartIdentifier tableAlias
- (PARTITION partition=identifierList)?
- (USING relation (COMMA relation)*)
+ | explain? cte? DELETE FROM tableName=multipartIdentifier
+ partitionSpec? tableAlias
+ (USING relation (COMMA relation)*)?
Review Comment:
why do we change the order of `tableAlias` and `partitionSpec`? the modified
syntax does not follow the documentation and introduces compatibility issues.
--
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.
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
