Re: [PERFORM] In progress INSERT wrecks plans on table

From: Josh Berkus <josh(at)agliodbs(dot)com>
To: Abhijit Menon-Sen <ams(at)2ndQuadrant(dot)com>
Cc: Simon Riggs <simon(at)2ndQuadrant(dot)com>, Mark Kirkwood <mark(dot)kirkwood(at)catalyst(dot)net(dot)nz>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Gavin Flower <gavinflower(at)archidevsys(dot)co(dot)nz>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: [PERFORM] In progress INSERT wrecks plans on table
Date: 2013-07-10 16:47:34
Message-ID: 51DD9026.2030107@agliodbs.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-performance

On 07/08/2013 10:11 AM, Josh Berkus wrote:
> On 06/23/2013 09:43 PM, Abhijit Menon-Sen wrote:
>> (Cc: to pgsql-performance dropped, pgsql-hackers added.)
>>
>> At 2013-05-06 09:14:01 +0100, simon(at)2ndQuadrant(dot)com wrote:
>>>
>>> New version of patch attached which fixes a few bugs.
>>
>> I read the patch, but only skimmed the earlier discussion about it. In
>> isolation, I can say that the patch applies cleanly and looks sensible
>> for what it does (i.e., cache pgprocno to speed up repeated calls to
>> TransactionIdIsInProgress(somexid)).
>>
>> In that sense, it's ready for committer, but I don't know if there's a
>> better/more complete/etc. way to address the original problem.
>
> Has this patch had performance testing? Because of the list crossover I
> don't have any information on that.

Due to the apparent lack of performance testing, I'm setting this back
to "needs review".

--
Josh Berkus
PostgreSQL Experts Inc.
http://pgexperts.com

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Benedikt Grundmann 2013-07-10 16:53:51 hardware donation
Previous Message Christopher Browne 2013-07-10 16:27:53 Re: Listen/notify across clusters

Browse pgsql-performance by date

  From Date Subject
Next Message Huang, Suya 2013-07-11 02:03:48 how to speed up the index creation in GP?
Previous Message Kevin Grittner 2013-07-10 13:37:33 Re: Process in state BIND, authentication, PARSE