cambyzju commented on code in PR #12262:
URL: https://github.com/apache/doris/pull/12262#discussion_r961885966
##########
fe/fe-core/src/main/cup/sql_parser.cup:
##########
@@ -4847,8 +4847,6 @@ type ::=
{: RESULT = ScalarType.createVarcharType(-1); :}
| KW_ARRAY LESSTHAN type:value_type GREATERTHAN
{: RESULT = new ArrayType(value_type); :}
- | KW_ARRAY LESSTHAN KW_NOT_NULL LPAREN type:value_type RPAREN GREATERTHAN
- {: RESULT = new ArrayType(value_type, false); :}
Review Comment:
Lots of codes who called `new ArrayType` will also need to catch the
exception, if we throw an exception while new ArrayType. I think just remove
these too lines is better.
--
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]
