This is an automated email from the ASF dual-hosted git repository.
morningman pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/master by this push:
new 266d54bb39 [Refine](sql block)unify exception catching for sql block
(#17879)
266d54bb39 is described below
commit 266d54bb39d35ade8ab529fce76db77ea8d78897
Author: GoGoWen <[email protected]>
AuthorDate: Sun Apr 9 20:09:16 2023 +0800
[Refine](sql block)unify exception catching for sql block (#17879)
currently, the AnalysisException throw by matchSql will catch immediately.
however, the AnalysisException throws by checkLimitations will catch as
UserException.
---
fe/fe-core/src/main/java/org/apache/doris/qe/StmtExecutor.java | 10 ++--------
1 file changed, 2 insertions(+), 8 deletions(-)
diff --git a/fe/fe-core/src/main/java/org/apache/doris/qe/StmtExecutor.java
b/fe/fe-core/src/main/java/org/apache/doris/qe/StmtExecutor.java
index 20a684836d..9592c2b832 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/qe/StmtExecutor.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/qe/StmtExecutor.java
@@ -464,14 +464,8 @@ public class StmtExecutor implements ProfileWriter {
}
private boolean checkBlockRules() throws AnalysisException {
- try {
- Env.getCurrentEnv().getSqlBlockRuleMgr().matchSql(
- originStmt.originStmt, context.getSqlHash(),
context.getQualifiedUser());
- } catch (AnalysisException e) {
- LOG.warn(e.getMessage());
- context.getState().setError(e.getMysqlErrorCode(), e.getMessage());
- return true;
- }
+ Env.getCurrentEnv().getSqlBlockRuleMgr().matchSql(
+ originStmt.originStmt, context.getSqlHash(),
context.getQualifiedUser());
// limitations: partition_num, tablet_num, cardinality
List<ScanNode> scanNodeList = planner.getScanNodes();
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
