This is an automated email from the ASF dual-hosted git repository.
yiguolei pushed a commit to branch branch-4.1
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/branch-4.1 by this push:
new c680bb7a785 branch-4.1: [fix](parser) isNull should under
primaryExpression #63619 (#64979)
c680bb7a785 is described below
commit c680bb7a7850fbdb8ada53654d99890ce6f80ea2
Author: morrySnow <[email protected]>
AuthorDate: Tue Jun 30 16:08:44 2026 +0800
branch-4.1: [fix](parser) isNull should under primaryExpression #63619
(#64979)
picked from master #63619
---
.../antlr4/org/apache/doris/nereids/DorisParser.g4 | 4 ++--
.../doris/nereids/parser/LogicalPlanBuilder.java | 4 ++--
.../doris/nereids/parser/NereidsParserTest.java | 24 ++++++++++++++++++++++
3 files changed, 28 insertions(+), 4 deletions(-)
diff --git a/fe/fe-core/src/main/antlr4/org/apache/doris/nereids/DorisParser.g4
b/fe/fe-core/src/main/antlr4/org/apache/doris/nereids/DorisParser.g4
index 2c99f81f023..a710576d1dd 100644
--- a/fe/fe-core/src/main/antlr4/org/apache/doris/nereids/DorisParser.g4
+++ b/fe/fe-core/src/main/antlr4/org/apache/doris/nereids/DorisParser.g4
@@ -1654,8 +1654,6 @@ lambdaExpression
booleanExpression
: LOGICALNOT booleanExpression
#logicalNot
| EXISTS LEFT_PAREN query RIGHT_PAREN
#exist
- | (ISNULL | IS_NULL_PRED) LEFT_PAREN valueExpression RIGHT_PAREN
#isnull
- | IS_NOT_NULL_PRED LEFT_PAREN valueExpression RIGHT_PAREN
#is_not_null_pred
| valueExpression predicate?
#predicated
| NOT booleanExpression
#logicalNot
| left=booleanExpression operator=(AND | LOGICALAND)
right=booleanExpression #logicalBinary
@@ -1733,6 +1731,8 @@ primaryExpression
| (SUBSTR | SUBSTRING | MID) LEFT_PAREN
expression FROM expression (FOR expression)? RIGHT_PAREN
#substring
| POSITION LEFT_PAREN expression IN expression RIGHT_PAREN
#position
+ | (ISNULL | IS_NULL_PRED) LEFT_PAREN expression RIGHT_PAREN
#isnull
+ | IS_NOT_NULL_PRED LEFT_PAREN expression RIGHT_PAREN
#is_not_null_pred
| functionCallExpression
#functionCall
| value=primaryExpression LEFT_BRACKET index=valueExpression RIGHT_BRACKET
#elementAt
| value=primaryExpression LEFT_BRACKET begin=valueExpression
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 0969b3cca01..f44fe31afde 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
@@ -5119,12 +5119,12 @@ public class LogicalPlanBuilder extends
DorisParserBaseVisitor<Object> {
@Override
public Expression visitIsnull(IsnullContext context) {
- return ParserUtils.withOrigin(context, () -> new
IsNull(typedVisit(context.valueExpression())));
+ return ParserUtils.withOrigin(context, () -> new
IsNull(typedVisit(context.expression())));
}
@Override
public Expression visitIs_not_null_pred(Is_not_null_predContext context) {
- return ParserUtils.withOrigin(context, () -> new Not(new
IsNull(typedVisit(context.valueExpression()))));
+ return ParserUtils.withOrigin(context, () -> new Not(new
IsNull(typedVisit(context.expression()))));
}
public List<Expression> withInList(PredicateContext ctx) {
diff --git
a/fe/fe-core/src/test/java/org/apache/doris/nereids/parser/NereidsParserTest.java
b/fe/fe-core/src/test/java/org/apache/doris/nereids/parser/NereidsParserTest.java
index a339ce6cf02..3b0c669049e 100644
---
a/fe/fe-core/src/test/java/org/apache/doris/nereids/parser/NereidsParserTest.java
+++
b/fe/fe-core/src/test/java/org/apache/doris/nereids/parser/NereidsParserTest.java
@@ -30,7 +30,10 @@ import org.apache.doris.nereids.exceptions.ParseException;
import org.apache.doris.nereids.exceptions.SyntaxParseException;
import org.apache.doris.nereids.glue.LogicalPlanAdapter;
import org.apache.doris.nereids.trees.expressions.Cast;
+import org.apache.doris.nereids.trees.expressions.EqualTo;
import org.apache.doris.nereids.trees.expressions.Expression;
+import org.apache.doris.nereids.trees.expressions.IsNull;
+import org.apache.doris.nereids.trees.expressions.Not;
import org.apache.doris.nereids.trees.expressions.OrderExpression;
import org.apache.doris.nereids.trees.expressions.functions.generator.Unnest;
import org.apache.doris.nereids.trees.expressions.literal.DecimalLiteral;
@@ -1642,4 +1645,25 @@ public class NereidsParserTest extends ParserTestBase {
plan = nereidsParser.parseSingle(sql);
Assertions.assertInstanceOf(UpdateCommand.class, plan);
}
+
+ @Test
+ public void testIsNullAndIsNotNullExpression() {
+ NereidsParser nereidsParser = new NereidsParser();
+ String sql = "ISNULL(X) = 1";
+ Expression expression = nereidsParser.parseExpression(sql);
+
+ Assertions.assertInstanceOf(EqualTo.class, expression);
+ Assertions.assertInstanceOf(IsNull.class, expression.child(0));
+
+ sql = "IS_NULL_PRED(X) = 1";
+ expression = nereidsParser.parseExpression(sql);
+ Assertions.assertInstanceOf(EqualTo.class, expression);
+ Assertions.assertInstanceOf(IsNull.class, expression.child(0));
+
+ sql = "IS_NOT_NULL_PRED(X) = 1";
+ expression = nereidsParser.parseExpression(sql);
+ Assertions.assertInstanceOf(EqualTo.class, expression);
+ Assertions.assertInstanceOf(Not.class, expression.child(0));
+ Assertions.assertInstanceOf(IsNull.class,
expression.child(0).child(0));
+ }
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
