References: <CAKU4AWoZksNZ4VR-fLTdwmiR91WU8qViDBNQKNwY=7iyo+uV0w@mail.gmail.com>
 <247851.1668293143@sss.pgh.pa.us>
 <CAKU4AWpp8aANvwU1ivjPtF3AakFjTf1jBm_OFmPSdCqizjBQLw@mail.gmail.com>
 <b9437269-0480-4016-91bf-61168f72bdd3@yandex.ru>
 <CAKU4AWrwggExWQ6mg7QGT59TVL6BGwBKbVn3ZQmmOAwOF3-w=g@mail.gmail.com>
 <602d6b44-c734-4f7f-8999-99b6cac9bf17@yandex.ru>
 <CAKU4AWphxS_QKEeUhSkt46_+7W8Y3ddr9q9roK4oWt0g2BZVmQ@mail.gmail.com>
 <00de47dd-7a38-4225-a47e-65a8ba6de8d3@yandex.ru>
 <CALDaNm2ck2ykEpF+7LzT6c1GeXLn-YgDBwUh4VmQiZmeHEUetQ@mail.gmail.com>
 <CAPpHfduAWZKpfe0Th+AJ4pGkJpBuSAgY5X0Xc0-XmUYOSYPm-g@mail.gmail.com>
User-agent: mu4e 1.10.7; emacs 29.1
From: Andy Fan <zhihuifan1213@163.com>
To: Alexander Korotkov <aekorotkov@gmail.com>
Cc: vignesh C <vignesh21@gmail.com>, Alena Rybakina
 <lena.ribackina@yandex.ru>, Andy Fan <zhihui.fan1213@gmail.com>,
 pgsql-hackers <pgsql-hackers@postgresql.org>, Tom Lane
 <tgl@sss.pgh.pa.us>, Richard Guo <guofenglinux@gmail.com>, Andrey Lepikhov
 <a.lepikhov@postgrespro.ru>, pgsql-hackers@lists.postgresql.org
Subject: Re: A new strategy for pull-up correlated ANY_SUBLINK
Date: Thu, 15 Feb 2024 17:51:28 +0800
In-reply-to: <CAPpHfduAWZKpfe0Th+AJ4pGkJpBuSAgY5X0Xc0-XmUYOSYPm-g@mail.gmail.com>
Message-ID: <871q9ehx66.fsf@163.com>
MIME-Version: 1.0
Content-Type: text/plain
Archived-At: <https://www.postgresql.org/message-id/871q9ehx66.fsf%40163.com>
Precedence: bulk


Hi Alexander,

> Hi!
>
>> If the changes of Alena are ok, can you merge the changes and post an
>> updated version so that CFBot can apply the patch and verify the
>> changes. As currently CFBot is trying to apply only Alena's changes
>> and failing with the following at [1]:
>
> I think this is a very nice and pretty simple optimization.  I've
> merged the changes by Alena, and slightly revised the code changes in
> convert_ANY_sublink_to_join().  I'm going to push this if there are no
> objections.

Thanks for picking up this! I double checked the patch, it looks good to
me. 

-- 
Best Regards
Andy Fan