Re: Performance issues with v18 SQL-language-function changes
Bruce Momjian <bruce@momjian.us>
From: Bruce Momjian <bruce@momjian.us>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Robert Haas <robertmhaas@gmail.com>, pgsql-hackers@lists.postgresql.org, Alexander Pyhalov <a.pyhalov@postgrespro.ru>, Pavel Stehule <pavel.stehule@gmail.com>
Date: 2025-04-16T18:53:38Z
Lists: pgsql-hackers
On Wed, Apr 16, 2025 at 01:43:46PM -0400, Tom Lane wrote: > Bruce Momjian <bruce@momjian.us> writes: > > On Mon, Apr 14, 2025 at 10:38:29AM -0400, Robert Haas wrote: > >> I agree that we should do something about this. I haven't reviewed > >> your patches but the approach sounds broadly reasonable. > > > Yep, we went down the road in PG 18 to convert syntax, and now we have > > to fix this, or we have to revert all the PG 18 syntax changes, which > > seems like a step backward. > > I'm confused? 0dca5d68d didn't have anything to do with > syntax changes, just with when planning happens. I was referencing the contrib initialization functions you converted to use SQL-standard function bodies: commit 68ff25eef12 Author: Tom Lane <tgl@sss.pgh.pa.us> Date: Sun Dec 29 14:58:05 2024 -0500 contrib/pageinspect: Use SQL-standard function bodies. In the same spirit as 969bbd0fa, 13e3796c9, 3f323eba8. Tom Lane and Ronan Dunklau Discussion: https://postgr.es/m/3316564.aeNJFYEL58@aivenlaptop I thought that was what you were saying were now slower; maybe I was confused. -- Bruce Momjian <bruce@momjian.us> https://momjian.us EDB https://enterprisedb.com Do not let urgent matters crowd out time for investment in the future.
Commits
-
Don't use a tuplestore if we don't have to for SQL-language functions.
- e83a8ae44729 18.0 landed
-
Change SQL-language functions to use the plan cache.
- 0dca5d68d7be 18.0 cited
-
contrib/pageinspect: Use SQL-standard function bodies.
- 68ff25eef125 18.0 cited