Re: [HACKERS] pgsql: Improve performance of SendRowDescriptionMessage.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Andres Freund <andres(at)anarazel(dot)de>
Cc: Noah Misch <noah(at)leadboat(dot)com>, pgsql-committers(at)postgresql(dot)org
Subject: Re: [HACKERS] pgsql: Improve performance of SendRowDescriptionMessage.
Date: 2017-10-13 18:19:22
Message-ID: 1820.1507918762@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

Andres Freund <andres(at)anarazel(dot)de> writes:
> A easiest way to do this would be to put something like
> CPPFLAGS="$CPPFLAGS -Dpg_restrict" into the existing
> if test "$GCC" != yes ; then
> block in a/src/template/linux. But that'd probably result in "macro
> redefined" warnings or somesuch.

You mean src/template/aix, no? Agreed, that seems like a reasonable
place to control it. But I'm pretty sure the above would flat out
not work, the #define in pg_config.h would override it.

> So it'd probably better to introduce a FORCE_DISABLE_RESTRICT=yes, set
> at the same place, that's then tested before running the relevant
> configure check?

+1. I think you don't actually have to skip the configure check,
and there might be some value in letting it carry on normally
(so that "restrict" is set properly). We'd just want it to affect
what pg_restrict gets defined as. Something like

if test "$ac_cv_c_restrict" = "no" -o "x$FORCE_DISABLE_RESTRICT" = "xyes"; then
pg_restrict=""
else
pg_restrict="$ac_cv_c_restrict"
fi

regards, tom lane

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Andres Freund 2017-10-13 18:24:05 Re: [HACKERS] pgsql: Improve performance of SendRowDescriptionMessage.
Previous Message Andres Freund 2017-10-13 18:01:06 Re: [HACKERS] pgsql: Improve performance of SendRowDescriptionMessage.

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2017-10-13 18:24:05 Re: [HACKERS] pgsql: Improve performance of SendRowDescriptionMessage.
Previous Message Andres Freund 2017-10-13 18:17:19 Re: Improve catcache/syscache performance.