yiguolei commented on code in PR #30817:
URL: https://github.com/apache/doris/pull/30817#discussion_r1477644281


##########
fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/commands/call/CallFunc.java:
##########
@@ -28,13 +29,14 @@ public abstract class CallFunc {
     /**
      * Get the instance of CallFunc
      */
-    public static CallFunc getFunc(UserIdentity user, UnboundFunction 
unboundFunction) {
+    public static CallFunc getFunc(ConnectContext ctx, UserIdentity user, 
UnboundFunction unboundFunction) {
         String funcName = unboundFunction.getName().toUpperCase();
         switch (funcName) {
-            case "EXECUTE_STMT":
+            // TODO, built-in functions require a separate management

Review Comment:
   always forward call procedure command to master because there maybe insert 
and select sql in one store procedure. Doris could not ensure the select sql 
could read the data inserted before. There maybe data consistency problem.



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