From 70a4e86062c0f5bbdaf582a5ed78834a7db2bee7 Mon Sep 17 00:00:00 2001 From: Jibing Li Date: Tue, 14 Nov 2023 20:11:57 +0800 Subject: [PATCH] Fix nereids fail to parse tablesample bug. --- .../org/apache/doris/nereids/parser/LogicalPlanBuilder.java | 2 +- fe/fe-core/src/main/java/org/apache/doris/qe/StmtExecutor.java | 3 +-- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/fe/fe-core/src/main/java/org/apache/doris/nereids/parser/LogicalPlanBuilder.java b/fe/fe-core/src/main/java/org/apache/doris/nereids/parser/LogicalPlanBuilder.java index eb4e5b3c1a3548..e899b5d0898c7a 100644 --- a/fe/fe-core/src/main/java/org/apache/doris/nereids/parser/LogicalPlanBuilder.java +++ b/fe/fe-core/src/main/java/org/apache/doris/nereids/parser/LogicalPlanBuilder.java @@ -2038,7 +2038,7 @@ public Object visitSample(SampleContext ctx) { return new TableSample(percent, true, seek); } SampleByRowsContext sampleByRowsContext = (SampleByRowsContext) sampleContext; - long rows = Long.parseLong(sampleByRowsContext.ROWS().getText()); + long rows = Long.parseLong(sampleByRowsContext.INTEGER_VALUE().getText()); return new TableSample(rows, false, seek); } 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 cad5da4aa6a15b..5ca0c3ae1a486c 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 @@ -2503,8 +2503,7 @@ public List executeInternalQuery() { planner = new NereidsPlanner(statementContext); planner.plan(parsedStmt, context.getSessionVariable().toThrift()); } catch (Exception e) { - LOG.warn("Arrow Flight SQL fall back to legacy planner, because: {}", - e.getMessage(), e); + LOG.warn("Fall back to legacy planner, because: {}", e.getMessage(), e); parsedStmt = null; planner = null; context.getState().setNereids(false);