Re: [COMMITTERS] pgsql: Improve performance ofSendRowDescriptionMessage. - Mailing list pgsql-committers

From Noah Misch
Subject Re: [COMMITTERS] pgsql: Improve performance ofSendRowDescriptionMessage.
Date
Msg-id 20171013070247.GA505228@rfd.leadboat.com
Whole thread Raw
In response to Re: [COMMITTERS] pgsql: Improve performance of SendRowDescriptionMessage.  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: [COMMITTERS] pgsql: Improve performance ofSendRowDescriptionMessage.  (Andres Freund <andres@anarazel.de>)
Re: [COMMITTERS] pgsql: Improve performance of SendRowDescriptionMessage.  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-committers
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@leadboat.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.

-- 
Sent via pgsql-committers mailing list (pgsql-committers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-committers

Attachment

pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: Re: [COMMITTERS] pgsql: Log diagnostic messages if errors occur during LDAP auth.
Next
From: Andres Freund
Date:
Subject: Re: [COMMITTERS] pgsql: Improve performance ofSendRowDescriptionMessage.