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