fix(planner): Fix stack overflow when applying RuleFilterPushDownJoin

[leiysky]

I hereby agree to the terms of the CLA available at: https://databend.rs/dev/policies/cla/

Summary

The root cause is RuleFilterPushDownJoin will sometimes generate duplicated conjunctions when there is OR expression.

This may cause the rule RuleEliminateFilter to be incorrectly applied to the optimized expression later, generating a new expression that would be handled by RuleFilterPushDownJoin again. Thus a dead loop occurs.

Closes #9236

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment

Name	Status	Preview	Comments	Updated
databend	⬜️ Ignored (Inspect)			Jan 17, 2023 at 8:37AM (UTC)

[BohuTANG]

[SQL] SELECT t0.actual_rowid, t6.c0, t0.c0 FROM t0 RIGHT JOIN t6 ON ((t0.actual_rowid)>=(((((t6.c0)<=(t6.c0))) IS NOT NULL)::INTEGER)) GROUP BY t0.actual_rowid, t6.c0, t0.c0 order by actual_rowid
[Diff] (-expected|+actual)
-   0 NULL 1969-12-16
-   1 NULL 1970-01-15
-   2 NULL 1970-01-11
+   0 NULL NULL
+   1 NULL NULL
+   2 NULL NULL
    3 NULL NULL
    4 NULL NULL
    5 NULL NULL
    6 NULL NULL
-   7 NULL 1970-01-23
+   7 NULL NULL
    8 NULL NULL
    9 NULL NULL
at tests/sqllogictests/suites/duckdb/issues/rigger/test_non_equality_right_join.test:46

Error: SelfError("sqllogictest failed")
Error: Process completed with exit code 1.