Re: pgsql: Improve performance of SendRowDescriptionMessage.

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

In an earlier message, I said I didn't see the problem with xlc 13.1.3. I
withdraw that statement. I had tested old code (commit c629324, 20 Aug),
rendering the test invalid. xlc 13.1.3 does break commit 91d5f1a, and
removing "restrict" fixes things as it did the older version.

On Thu, Oct 12, 2017 at 10:44:20PM -0400, Tom Lane wrote:
> Noah Misch <noah(at)leadboat(dot)com> writes:
> > On Thu, Oct 12, 2017 at 04:08:44PM -0700, Andres Freund wrote:
> >> Noah, any chance you could force restrict to off on that animal?
>
> > I can confirm it allows "make check" to pass.
>
> So that leaves us with two theories:
>
> 1. hornet's compiler contains a bug that causes it to misoptimize
> in the presence of "restrict".
>
> 2. There's a bug in the way HEAD is applying "restrict", which happens
> not to manifest on other platforms.
>
> While I have to agree with Andres' evident feeling that it's probably
> #1, I do not think we should dismiss #2 without inquiring a bit
> harder. It would be really useful, I think, if we could characterize
> exactly how the RowDescription output is broken in that build.
> Noah, could you capture some of those messages somehow?

I hacked psql to call PQtrace() and ran "psql -Xc 'select true'" in the
defective configuration and in a working x64 GNU/Linux configuration. I've
attached both PQtrace() products.

Attachment Content-Type Size
extraneous-xlc-aix.pqtrace text/plain 551 bytes
normal-gcc-gnu.pqtrace text/plain 594 bytes

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Andres Freund 2017-10-13 07:32:43 Re: pgsql: Improve performance of SendRowDescriptionMessage.
Previous Message Tom Lane 2017-10-13 03:52:28 Re: pgsql: Log diagnostic messages if errors occur during LDAP auth.

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2017-10-13 07:32:43 Re: pgsql: Improve performance of SendRowDescriptionMessage.
Previous Message Jeevan Chalke 2017-10-13 06:36:23 Re: Partition-wise aggregation/grouping