Baymine opened a new pull request, #65254:
URL: https://github.com/apache/doris/pull/65254

   ### What problem does this PR solve?
   
   Issue Number: close #65252
   
   Problem Summary:
   
   `DUMP` is a lexer keyword (used by `PLAN REPLAYER DUMP`) but was missing 
from the `nonReserved` list in `DorisParser.g4`, so `dump` could not be used as 
a regular identifier (column, alias, table alias, CTE name, function argument, 
qualified reference). This restores backward compatibility with older versions 
(e.g. 2.1) where `dump` was usable as an identifier. Adding `DUMP` to 
`nonReserved` keeps it usable both as the `PLAN REPLAYER DUMP` keyword and as 
an identifier.
   
   ### Release note
   
   None
   
   ### Check List (For Author)
   
   - Test
       - [x] Unit Test — added `NereidsParserTest#testParseDumpAsIdentifier` 
covering `dump` as column alias, column ref, table alias, WHERE clause, 
qualified ref, function arg, CTE name, subquery alias, and confirming `PLAN 
REPLAYER DUMP` still parses.
   - Behavior changed:
       - [x] No.
   - Does this need documentation?
       - [x] No.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to