MIME-Version: 1.0
References: <CAMbWs4_X1mN=ic+SxcyymUqFx9bB8pqSLTGJ-F=MHy4PW3eRXw@mail.gmail.com>
 <CAMbWs4_ccjTJSksy5Z_WxmhoxO5n2PZ49qSQHaXb2JxzKZ=x8g@mail.gmail.com> <CAMbWs48G3SP5HC7xXg7vPaRPBjPQkbdOyYxFQRHJzte+0+vyyQ@mail.gmail.com>
In-Reply-To: <CAMbWs48G3SP5HC7xXg7vPaRPBjPQkbdOyYxFQRHJzte+0+vyyQ@mail.gmail.com>
From: vignesh C <vignesh21@gmail.com>
Date: Sun, 7 Jan 2024 12:33:00 +0530
Message-ID: <CALDaNm3uQWo7zj2KTtd9iH_yEVCToTGXDjay_9Jk-EmMf+_kAQ@mail.gmail.com>
Subject: Re: Support "Right Semi Join" plan shapes
To: Richard Guo <guofenglinux@gmail.com>
Cc: PostgreSQL-development <pgsql-hackers@postgresql.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Archived-At: <https://www.postgresql.org/message-id/CALDaNm3uQWo7zj2KTtd9iH_yEVCToTGXDjay_9Jk-EmMf%2B_kAQ%40mail.gmail.com>
Precedence: bulk

On Wed, 1 Nov 2023 at 11:25, Richard Guo <guofenglinux@gmail.com> wrote:
>
>
> On Thu, Aug 10, 2023 at 3:24=E2=80=AFPM Richard Guo <guofenglinux@gmail.c=
om> wrote:
>>
>> The cfbot reminds that this patch does not apply any more, so rebase it
>> to v2.
>
>
> Attached is another rebase over the latest master.  Any feedback is
> appreciated.

One of the tests in CFBot has failed at [1] with:
-   Relations: (public.ft1 t1) SEMI JOIN (public.ft2 t2)
-   Remote SQL: SELECT r1."C 1", r1.c2, r1.c3, r1.c4, r1.c5, r1.c6,
r1.c7, r1.c8 FROM "S 1"."T 1" r1 WHERE ((r1."C 1" < 20)) AND EXISTS
(SELECT NULL FROM "S 1"."T 1" r3 WHERE ((r3."C 1" > 10)) AND
((date(r3.c5) =3D '1970-01-17'::date)) AND ((r3.c3 =3D r1.c3))) ORDER BY
r1."C 1" ASC NULLS LAST
-(4 rows)
+   Sort Key: t1.c1
+   ->  Foreign Scan
+         Output: t1.c1, t1.c2, t1.c3, t1.c4, t1.c5, t1.c6, t1.c7, t1.c8
+         Relations: (public.ft1 t1) SEMI JOIN (public.ft2 t2)
+         Remote SQL: SELECT r1."C 1", r1.c2, r1.c3, r1.c4, r1.c5,
r1.c6, r1.c7, r1.c8 FROM "S 1"."T 1" r1 WHERE ((r1."C 1" < 20)) AND
EXISTS (SELECT NULL FROM "S 1"."T 1" r3 WHERE ((r3."C 1" > 10)) AND
((date(r3.c5) =3D '1970-01-17'::date)) AND ((r3.c3 =3D r1.c3)))
+(7 rows)

More details are available at [2].

[1] - https://cirrus-ci.com/task/4868751326183424
[2] - https://api.cirrus-ci.com/v1/artifact/task/4868751326183424/testrun/b=
uild/testrun/postgres_fdw/regress/regression.diffs

Regards,
Vignesh