Re: Removing unneeded self joins

Hi,

On 2022-10-05 17:25:18 +0500, Andrey Lepikhov wrote:
> New version, rebased onto current master.
> Nothing special, just rebase.

This doesn't pass the main regression tests due to a plan difference.
https://cirrus-ci.com/task/5537518245380096
https://api.cirrus-ci.com/v1/artifact/task/5537518245380096/testrun/build/testrun/regress/regress/regression.diffs

diff -U3 /tmp/cirrus-ci-build/src/test/regress/expected/join.out
/tmp/cirrus-ci-build/build/testrun/regress/regress/results/join.out
--- /tmp/cirrus-ci-build/src/test/regress/expected/join.out    2022-12-05 19:11:52.453920838 +0000
+++ /tmp/cirrus-ci-build/build/testrun/regress/regress/results/join.out    2022-12-05 19:15:21.864183651 +0000
@@ -5806,7 +5806,7 @@
  Nested Loop
    Join Filter: (sj_t3.id = sj_t1.id)
    ->  Nested Loop
-         Join Filter: (sj_t3.id = sj_t2.id)
+         Join Filter: (sj_t2.id = sj_t3.id)
          ->  Nested Loop Semi Join
                ->  Nested Loop
                      ->  HashAggregate

Greetings,

Andres Freund