On 4/24/19 5:10 PM, Peter Geoghegan wrote: > On Wed, Apr 24, 2019 at 2:07 PM Joe Conway <m...@joeconway.com> wrote: >> I just noticed that TRACE_SORT is defined by default (since 2005 >> apparently). It seems odd since it is the only debugging code enabled by >> default. > > I think that we should get rid of the #ifdef stuff, so that it isn't > possible to disable the trace_sort instrumentation my commenting out > the TRACE_SORT entry in pg_config_manual.h. I recall being opposed on > this point by Robert Haas. Possibly because he just didn't want to > deal with it at the time.
Has anyone ever (or recently) measured the impact on performance to have this enabled? Is it that generically useful for debugging of production instances of Postgres that we really want it always enabled despite the performance impact? Maybe the answer to both is "yes", but if so I would agree that we ought to remove the define and ifdef's and just bake it in. Joe -- Crunchy Data - http://crunchydata.com PostgreSQL Support for Secure Enterprises Consulting, Training, & Open Source Development
signature.asc
Description: OpenPGP digital signature