morningman commented on code in PR #11338:
URL: https://github.com/apache/doris/pull/11338#discussion_r933099724
##########
fe/fe-core/src/main/java/org/apache/doris/analysis/UseStmt.java:
##########
@@ -35,19 +35,35 @@
*/
public class UseStmt extends StatementBase {
private static final Logger LOG = LogManager.getLogger(UseStmt.class);
+ private String catalogName;
private String database;
public UseStmt(String db) {
database = db;
}
+ public UseStmt(String catalogName, String db) {
Review Comment:
No need to modify use stmt in `sql_parser.cup`?
##########
fe/fe-core/src/main/cup/sql_parser.cup:
##########
@@ -2738,6 +2742,10 @@ show_param ::=
{:
RESULT = new ShowDbStmt(parser.wild, parser.where);
:}
+ | KW_SCHEMAS KW_FROM STRING_LITERAL:catalogName
Review Comment:
use `ident` instead of `STRING_LITERAL`. So we don't need to use `""`.
--
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]
