MIME-Version: 1.0
References: 
 <CAPpHfdvLCgXgdhSvuPCkek+r=P4E-WiNceWfh7UQztiMzfFcvg@mail.gmail.com>
 <aegLPbRmc95UbOZG@paquier.xyz>
 <CAPpHfdsOkkeobCfO9hJvaE2fRcdGmWJoX0umx--DnApmtmefCg@mail.gmail.com>
 <CAJDiXgjNySH7Od3nZGjWy30sKv+-12c=kajL7Qfkj3y+CDRwtA@mail.gmail.com>
 <CAPpHfds8pwuwF69JFs0SQz58op-U4ddR3KWv0NVEij2mpkzgpw@mail.gmail.com>
 <239d181d-1415-49ee-ad57-b307f1a7ba66@uni-muenster.de>
 <CAJDiXgi_SW3DVvXb4+n04NA1YfeeXmT3gUnSG7h4JwBd+RdJGg@mail.gmail.com>
 <0e666e40-d003-4c92-95cf-1a33a647401f@uni-muenster.de>
 <CAJDiXgiyy+GG6a5=nDPQeOQTjdQo53+6N0rAzZ=M3baWGPJNsA@mail.gmail.com>
 <CAPpHfdsS49OTo2Af2GeoKJD-1Dy-hOzbje_AWOa_bw5Q9kEi7w@mail.gmail.com>
 <af2AdMw3HjXgkzWJ@paquier.xyz>
In-Reply-To: <af2AdMw3HjXgkzWJ@paquier.xyz>
From: Alexander Korotkov <aekorotkov@gmail.com>
Date: Fri, 8 May 2026 09:39:09 +0300
Message-ID: 
 <CAPpHfduFytN4edJnJUh-D+CyLBbhab4-HfKhOoftZ_TXUfGH5g@mail.gmail.com>
Subject: Re: Fix bug with accessing to temporary tables of other sessions
To: Michael Paquier <michael@paquier.xyz>
Cc: Daniil Davydov <3danissimo@gmail.com>,
 Jim Jones <jim.jones@uni-muenster.de>,
	Soumya S Murali <soumyamurali.work@gmail.com>, Tom Lane <tgl@sss.pgh.pa.us>,
	Stepan Neretin <slpmcf@gmail.com>,
 PostgreSQL Hackers <pgsql-hackers@postgresql.org>,
	Mohamed Ali <moali.pg@gmail.com>, Nazneen Jafri <jafrinazneen@gmail.com>,
	Shawn McCoy <shawn.the.mccoy@gmail.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Archived-At: 
 <https://www.postgresql.org/message-id/CAPpHfduFytN4edJnJUh-D%2BCyLBbhab4-HfKhOoftZ_TXUfGH5g%40mail.gmail.com>
Precedence: bulk

On Fri, May 8, 2026 at 9:19=E2=80=AFAM Michael Paquier <michael@paquier.xyz=
> wrote:
> On Thu, May 07, 2026 at 11:04:16AM +0300, Alexander Korotkov wrote:
> > Let me do a quick summary:
> >  * Our buffer manager is not capable for reading temp tables of other s=
essions.
> >  * This was covered by explicit checks, but broken since b7b0f3f27241
> > introduced alternative code path for reading tables.
> >  * This doesn't apply to DROP TABLE.  DROP TABLE is a conscious
> > exclusion and the only operation we can do correctly for other
> > session' temp tables.  There is an explicit exclusion in the code to
> > skip the attempt to cleanup buffers of other session' temp tables.
> >  * This patchset consists of tests (0001) for various operations with
> > other session's temp tables including buggy behavior, and the fix
> > (0002) including changes for tests.
> >
> > Thus, I don't see the reason why this shouldn't be committed and
> > backpatched to PG17 (first release containing b7b0f3f27241).
> > Opinions?  Michael?
>
> Hmm.  I don't have any counter-arguments against a backpatch based on
> your argument related to b7b0f3f27241.  Thanks for reorganizing the
> patch set so as the tests happen first, and the changes in the code
> become second.
>
> If you wish me to look at this patch set in details, I may be able to
> do so around the beginning of next week.  I'm not sure that there is a
> strong urgency in tackling this issue for this minor release, this
> could wait a bit more..

Absolutely, no urgency to include it into this minor release as I
already agreed [1].  You're very welcome if you could take a look in
the beginning of the next week.

Links.
1. https://www.postgresql.org/message-id/CAPpHfdsEr6RF-SkzVGD6PzFusENbpNPxK=
nRY7iQQ%3DcZQYJia6w%40mail.gmail.com

------
Regards,
Alexander Korotkov
Supabase