xiedeyantu commented on code in PR #48715:
URL: https://github.com/apache/doris/pull/48715#discussion_r1986508791


##########
fe/fe-core/src/main/java/org/apache/doris/nereids/parser/LogicalPlanBuilder.java:
##########
@@ -5927,6 +5928,34 @@ public LogicalPlan 
visitShowQueuedAnalyzeJobs(ShowQueuedAnalyzeJobsContext ctx)
         return new ShowQueuedAnalyzeJobsCommand(tableName, stateKey, 
stateValue);
     }
 
+    @Override
+    public LogicalPlan visitShowTableStatus(DorisParser.ShowTableStatusContext 
ctx) {
+        String ctlName = null;
+        String dbName = null;
+        if (ctx.database != null) {
+            List<String> nameParts = visitMultipartIdentifier(ctx.database);
+            if (nameParts.size() == 1) {
+                dbName = nameParts.get(0);
+            } else if (nameParts.size() == 2) {
+                ctlName = nameParts.get(0);
+                dbName = nameParts.get(1);
+            } else {

Review Comment:
   > how about cluster name?
   
   Would it be better to create a `DbName` ? `ClusterName` may be a bit 
ambiguous in understanding the code, although ClusterName can be used here.



-- 
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

Reply via email to