Message-ID: <dc465c1a-c9dc-4f57-af73-9c5f1cd0854a@postgrespro.ru>
Date: Wed, 21 Feb 2024 14:59:45 +0700
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: Removing unneeded self joins
Content-Language: en-US
To: Richard Guo <guofenglinux@gmail.com>
Cc: Alexander Korotkov <aekorotkov@gmail.com>,
 Alexander Lakhin <exclusion@gmail.com>,
 "Gregory Stark (as CFM)" <stark.cfm@gmail.com>,
 =?UTF-8?Q?Micha=C5=82_K=C5=82eczek?= <michal@kleczek.org>,
 PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>,
 Tom Lane <tgl@sss.pgh.pa.us>
References: <64486b0b-0404-e39e-322d-0801154901f3@postgrespro.ru>
 <922623af-872e-4cdc-9238-129b18bee42d@postgrespro.ru>
 <CAPpHfduoAHTp6O7Q8PDe_w=30v0n_06pGe5M65iR4DYcfaYnRw@mail.gmail.com>
 <107682e0-137a-d0fb-2733-7fc6e9562690@gmail.com>
 <CAPpHfduLxYm4biJrTbjBxTAW6vkxBswuQ2B=gXU+c37QJd6+Ow@mail.gmail.com>
 <a89f480f-8143-0965-f22d-0a892777f501@gmail.com>
 <0c5f461f-e26f-4fd9-9700-91b0e9c38fd6@postgrespro.ru>
 <336c748b-4ffe-5915-3d3c-ece6d19e021c@gmail.com>
 <4788f781-31bd-9796-d7d6-588a751c8787@gmail.com>
 <CAPpHfduYKqbWWT=YuBJyoExrwsJSgjUuJN1Gsr2YZMG=nh_ctQ@mail.gmail.com>
 <CAPpHfdvu+wyazT+ujJbJ5rKZrHQoG+FCVeWENA4pyf_A1W1WAA@mail.gmail.com>
 <56ee4520-e9d1-d519-54fe-c8bff880ce9b@gmail.com>
 <CAPpHfdsJ=MZNFw0L2T_k-shOk+4dPNx9j5RZzicy3NW+yiiQVg@mail.gmail.com>
 <CAPpHfdtB0A1Rz8k5yPMCM6ngLt2JM4NSaG8rF0=wT-txdcEf3g@mail.gmail.com>
 <5063f378-ca3c-405e-b1e8-1927ea9052e8@postgrespro.ru>
 <CAMbWs4-=PO6Mm9gNnySbx0VHyXjgnnYYwbN9dth=TLQweZ-M+g@mail.gmail.com>
From: Andrei Lepikhov <a.lepikhov@postgrespro.ru>
Organization: Postgres Professional
In-Reply-To: 
 <CAMbWs4-=PO6Mm9gNnySbx0VHyXjgnnYYwbN9dth=TLQweZ-M+g@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
Archived-At: 
 <https://www.postgresql.org/message-id/dc465c1a-c9dc-4f57-af73-9c5f1cd0854a%40postgrespro.ru>
Precedence: bulk

On 21/2/2024 14:26, Richard Guo wrote:
> This patch also includes some cosmetic tweaks for SJE test cases.  It
> does not change the test cases using catalog tables though.  I think
> that should be a seperate patch.
Thanks for this catch, it is really messy thing, keeping aside the 
question why we need two different subtrees for the same query.
I will look into your fix.

-- 
regards,
Andrei Lepikhov
Postgres Professional