Hi Richard
Thank you so much for your tireless work on this,I see the new version
of the patch improves some of the comments .I think it can commit in July
Thanks
On Thu, 25 Apr 2024 at 11:28, Richard Guo wrote:
> Here is another rebase with a commit message to help review. I also
>
Here is another rebase with a commit message to help review. I also
tweaked some comments.
Thanks
Richard
v5-0001-Support-Right-Semi-Join-plan-shapes.patch
Description: Binary data
On 06.03.2024 05:23, wenhui qiu wrote:
Hi Alena Rybakina
For merge join
+ /*
+ * For now we do not support RIGHT_SEMI join in mergejoin.
+ */
+ if (jointype == JOIN_RIGHT_SEMI)
+ {
+ *mergejoin_allowed = false;
+ return NIL;
+ }
Hi Alena Rybakina
For merge join
+ /*
+ * For now we do not support RIGHT_SEMI join in mergejoin.
+ */
+ if (jointype == JOIN_RIGHT_SEMI)
+ {
+ *mergejoin_allowed = false;
+ return NIL;
+ }
+
Tanks
On Wed, 6 Mar 2024 at 04:10, Alena Rybakina
wrote:
> To be honest, I didn't see it in the code,
To be honest, I didn't see it in the code, could you tell me where they
are, please?
On 05.03.2024 05:44, Richard Guo wrote:
On Tue, Jan 30, 2024 at 2:51 PM Alena Rybakina
wrote:
I have reviewed your patch and I think it is better to add an
Assert for
JOIN_RIGHT_SEMI to the
Hi Richard
Agree +1 ,I think can push now.
Richard
On Tue, 5 Mar 2024 at 10:44, Richard Guo wrote:
>
> On Tue, Jan 30, 2024 at 2:51 PM Alena Rybakina
> wrote:
>
>> I have reviewed your patch and I think it is better to add an Assert for
>> JOIN_RIGHT_SEMI to the ExecMergeJoin and
On Tue, Jan 30, 2024 at 2:51 PM Alena Rybakina
wrote:
> I have reviewed your patch and I think it is better to add an Assert for
> JOIN_RIGHT_SEMI to the ExecMergeJoin and ExecNestLoop functions to
> prevent the use of RIGHT_SEMI for these types of connections (NestedLoop
> and MergeJoin).
On Mon, Mar 4, 2024 at 10:33 AM wenhui qiu wrote:
> HI Richard
> Now it is starting the last commitfest for v17, can you respond to
> Alena Rybakina points?
>
Thanks for reminding. Will do that soon.
Thanks
Richard
HI Richard
Now it is starting the last commitfest for v17, can you respond to
Alena Rybakina points?
Regards
On Thu, 8 Feb 2024 at 13:50, wenhui qiu wrote:
> Hi Alena Rybakina
> I saw this code snippet also disable mergejoin ,I think it same effect
> + /*
> + * For now we do not
Hi Alena Rybakina
I saw this code snippet also disable mergejoin ,I think it same effect
+ /*
+ * For now we do not support RIGHT_SEMI join in mergejoin.
+ */
+ if (jointype == JOIN_RIGHT_SEMI)
+ {
+ *mergejoin_allowed = false;
+ return NIL;
+ }
+
Regards
Alena Rybakina 于2024年1月30日周二 14:51写道：
Hi! Thank you for your work on this subject.
I have reviewed your patch and I think it is better to add an Assert for
JOIN_RIGHT_SEMI to the ExecMergeJoin and ExecNestLoop functions to
prevent the use of RIGHT_SEMI for these types of connections (NestedLoop
and MergeJoin).
Mostly I'm
Hi vignesh C
Many thanks, I have marked it to "ready for committer"
Best wish
vignesh C 于2024年1月23日周二 10:56写道：
> On Mon, 22 Jan 2024 at 11:27, wenhui qiu wrote:
> >
> > Hi vignesh CI saw this path has been passed (
> https://cirrus-ci.com/build/6109321080078336),can we push it?
>
>
On Mon, 22 Jan 2024 at 11:27, wenhui qiu wrote:
>
> Hi vignesh CI saw this path has been passed
> (https://cirrus-ci.com/build/6109321080078336),can we push it?
If you have found no comments from your review and testing, let's mark
it as "ready for committer".
Regards,
Vignesh
Hi vignesh CI saw this path has been passed (
https://cirrus-ci.com/build/6109321080078336),can we push it?
Best wish
Richard Guo 于2024年1月9日周二 18:49写道：
>
> On Sun, Jan 7, 2024 at 3:03 PM vignesh C wrote:
>
>> One of the tests in CFBot has failed at [1] with:
>> - Relations: (public.ft1
On Sun, Jan 7, 2024 at 3:03 PM vignesh C wrote:
> 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
On Wed, 1 Nov 2023 at 11:25, Richard Guo wrote:
>
>
> On Thu, Aug 10, 2023 at 3:24 PM Richard Guo 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
Hi Richard Guo
I did a simple test ,Subqueries of type (in) can be supported, There
is a test sql that doesn't support it, and I think that's because it can't
pull up the subqueries.
```
test=# explain (costs off) SELECT t1.* FROM prt1_adv t1 WHERE EXISTS
(SELECT 1 FROM prt2_adv t2 WHERE
Hi Richard Guo I see that the test samples are all (exists)
subqueries ,I think semi join should also support ( in) and ( any)
subqueries. would you do more test on ( in) and ( any) subqueries?
Best whish
On Thu, Aug 10, 2023 at 3:24 PM Richard Guo 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.
Thanks
Richard
v3-0001-Support-Right-Semi-Join-plan-shapes.patch
On Tue, Apr 18, 2023 at 5:07 PM Richard Guo wrote:
> In thread [1] which discussed 'Right Anti Join', Tom once mentioned
> 'Right Semi Join'. After a preliminary investigation I think it is
> beneficial and can be implemented with very short change. With 'Right
> Semi Join', what we want to do
In thread [1] which discussed 'Right Anti Join', Tom once mentioned
'Right Semi Join'. After a preliminary investigation I think it is
beneficial and can be implemented with very short change. With 'Right
Semi Join', what we want to do is to just have the first match for each
inner tuple. For
21 matches
Mail list logo
