arthur-zhang commented on pull request #6150: URL: https://github.com/apache/incubator-doris/pull/6150#issuecomment-877642865
> Thanks for your PR. This is indeed a problem. > But I think it could be better to handle this situation in `StmtExecutor.handleQueryStmt()`. > Because here we have analyzed the statement, and all constant exprs has been calculated, so that we can simple check > if there is no table ref in statement and all select items are constant literal, and there is no alive backend, then return the result by FE. > This can cover more situation than only handle `select version()` and `select @@variables`; Good suggestion，I will move the code to `handleQueryStmt` and make it more general -- 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]
