Thread
Commits
-
Remove TRACE_SORT macro
- c8e2d422fd55 18.0 landed
-
Remove TRACE_SORT macro?
Peter Eisentraut <peter@eisentraut.org> — 2024-08-07T06:56:30Z
I think we could remove the TRACE_SORT macro. The TRACE_SORT macro has guarded the availability of the trace_sort GUC setting. But it has been enabled by default ever since it was introduced in PostgreSQL 8.1, and there have been no reports that someone wanted to disable it. So I think we could just remove the macro to simplify things.
-
Re: Remove TRACE_SORT macro?
Heikki Linnakangas <hlinnaka@iki.fi> — 2024-08-07T07:24:50Z
On 07/08/2024 09:56, Peter Eisentraut wrote: > I think we could remove the TRACE_SORT macro. > > The TRACE_SORT macro has guarded the availability of the trace_sort GUC > setting. But it has been enabled by default ever since it was > introduced in PostgreSQL 8.1, and there have been no reports that > someone wanted to disable it. So I think we could just remove the macro > to simplify things. +1, I don't see why anyone would want to build without TRACE_SORT. -- Heikki Linnakangas Neon (https://neon.tech)
-
Re: Remove TRACE_SORT macro?
Peter Eisentraut <peter@eisentraut.org> — 2024-08-14T06:14:22Z
On 07.08.24 09:24, Heikki Linnakangas wrote: > On 07/08/2024 09:56, Peter Eisentraut wrote: >> I think we could remove the TRACE_SORT macro. >> >> The TRACE_SORT macro has guarded the availability of the trace_sort GUC >> setting. But it has been enabled by default ever since it was >> introduced in PostgreSQL 8.1, and there have been no reports that >> someone wanted to disable it. So I think we could just remove the >> macro to simplify things. > > +1, I don't see why anyone would want to build without TRACE_SORT. committed