[fix](Planner): don't push down isNull predicate into view (apache#26288

)

We can't push down `isNull` predicate into view

fe/fe-core/src/main/java/org/apache/doris/analysis/BinaryPredicate.java

@@ -799,8 +799,8 @@ public static BinaryPredicate read(DataInput in) throws IOException {
  }
 
  @Override
- public Expr getResultValue(boolean inView) throws AnalysisException {
- recursiveResetChildrenResult(inView);
+ public Expr getResultValue(boolean forPushDownPredicatesToView) throws AnalysisException {
+ recursiveResetChildrenResult(forPushDownPredicatesToView);
  final Expr leftChildValue = getChild(0);
  final Expr rightChildValue = getChild(1);
  if (!(leftChildValue instanceof LiteralExpr)

fe/fe-core/src/main/java/org/apache/doris/analysis/CastExpr.java

@@ -399,8 +399,8 @@ public boolean canHashPartition() {
  }
 
  @Override
- public Expr getResultValue(boolean inView) throws AnalysisException {
- recursiveResetChildrenResult(inView);
+ public Expr getResultValue(boolean forPushDownPredicatesToView) throws AnalysisException {
+ recursiveResetChildrenResult(forPushDownPredicatesToView);
  final Expr value = children.get(0);
  if (!(value instanceof LiteralExpr)) {
  return this;

fe/fe-core/src/main/java/org/apache/doris/analysis/CompoundPredicate.java

@@ -230,8 +230,8 @@ public static boolean isOr(Expr expr) {
  }
 
  @Override
- public Expr getResultValue(boolean inView) throws AnalysisException {
- recursiveResetChildrenResult(inView);
+ public Expr getResultValue(boolean forPushDownPredicatesToView) throws AnalysisException {
+ recursiveResetChildrenResult(forPushDownPredicatesToView);
  boolean compoundResult = false;
  if (op == Operator.NOT) {
  final Expr childValue = getChild(0);

fe/fe-core/src/main/java/org/apache/doris/analysis/Expr.java

@@ -2194,10 +2194,10 @@ public boolean supportSerializable() {
  }
 
 
- protected void recursiveResetChildrenResult(boolean inView) throws AnalysisException {
+ protected void recursiveResetChildrenResult(boolean forPushDownPredicatesToView) throws AnalysisException {
  for (int i = 0; i < children.size(); i++) {
  final Expr child = children.get(i);
- final Expr newChild = child.getResultValue(inView);
+ final Expr newChild = child.getResultValue(forPushDownPredicatesToView);
  if (newChild != child) {
  setChild(i, newChild);
  }

fe/fe-core/src/main/java/org/apache/doris/analysis/InPredicate.java

@@ -308,8 +308,8 @@ public String toString() {
  }
 
  @Override
- public Expr getResultValue(boolean inView) throws AnalysisException {
- recursiveResetChildrenResult(inView);
+ public Expr getResultValue(boolean forPushDownPredicatesToView) throws AnalysisException {
+ recursiveResetChildrenResult(forPushDownPredicatesToView);
  final Expr leftChildValue = getChild(0);
  if (!(leftChildValue instanceof LiteralExpr) || !isLiteralChildren()) {
  return this;

fe/fe-core/src/main/java/org/apache/doris/analysis/IsNullPredicate.java

@@ -155,8 +155,10 @@ public boolean isNullable() {
  * fix issue 6390
  */
  @Override
- public Expr getResultValue(boolean inView) throws AnalysisException {
- recursiveResetChildrenResult(inView);
+ public Expr getResultValue(boolean forPushDownPredicatesToView) throws AnalysisException {
+ // Don't push down predicate to view for is null predicate because the value can contain null
+ // after outer join
+ recursiveResetChildrenResult(!forPushDownPredicatesToView);
  final Expr childValue = getChild(0);
  if (!(childValue instanceof LiteralExpr)) {
  return this;

fe/fe-core/src/main/java/org/apache/doris/analysis/SlotRef.java

@@ -617,8 +617,8 @@ public boolean matchExprs(List<Expr> exprs, SelectStmt stmt, boolean ignoreAlias
  }
 
  @Override
- public Expr getResultValue(boolean foldSlot) throws AnalysisException {
- if (!foldSlot) {
+ public Expr getResultValue(boolean forPushDownPredicatesToView) throws AnalysisException {
+ if (!forPushDownPredicatesToView) {
  return this;
  }
  if (!isConstant() || desc == null) {
@@ -630,7 +630,7 @@ public Expr getResultValue(boolean foldSlot) throws AnalysisException {
  }
  Expr expr = exprs.get(0);
  if (expr instanceof SlotRef) {
- return expr.getResultValue(foldSlot);
+ return expr.getResultValue(forPushDownPredicatesToView);
  }
  if (expr.isConstant()) {
  return expr;

fe/fe-core/src/main/java/org/apache/doris/analysis/VariableExpr.java

@@ -137,7 +137,7 @@ public Expr getLiteralExpr() {
  }
 
  @Override
- public Expr getResultValue(boolean inView) throws AnalysisException {
+ public Expr getResultValue(boolean forPushDownPredicatesToView) throws AnalysisException {
  if (!Strings.isNullOrEmpty(name) && VariableVarConverters.hasConverter(name)) {
  // Return the string type here so that it can correctly match the subsequent function signature.
  // And we also set `beConverted` to session variable name in StringLiteral, so that it can be cast back

regression-test/data/query_p0/literal_view/lietral_test.out

@@ -1,5 +1,10 @@
 -- This file is automatically generated. You should know what you did if you want to edit this
 -- !sql --
 
+-- !left --
+1 doris 10 \N
+2 spark 2 \N
+3 flink 20 \N
+
 -- !sql1 --
 

regression-test/suites/query_p0/literal_view/lietral_test.groovy

@@ -106,6 +106,15 @@ suite("literal_view_test") {
  insert into test_insert values (1,'doris',10),(2,'spark',2),(3,'flink',20);
  """
 
+ sql "set enable_nereids_planner=false"
+ order_qt_left """select * 
+ from test_insert 
+ left join (select 1 as v1) t1 
+ on false 
+ where t1.v1 is null
+ """
+ sql "set enable_nereids_planner=true"
+
  qt_sql1 """
  select id, name
  from (