Re: executor relation handling

From: Jesper Pedersen <jesper(dot)pedersen(at)redhat(dot)com>
To: Amit Langote <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp>, David Rowley <david(dot)rowley(at)2ndquadrant(dot)com>
Cc: Amit Langote <amitlangote09(at)gmail(dot)com>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Subject: Re: executor relation handling
Date: 2018-09-28 13:51:47
Message-ID: ad53a55a-cc1f-c38a-8735-7c40a5261339@redhat.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

On 9/28/18 4:58 AM, Amit Langote wrote:
>>> Okay, I've revised the text in the attached updated patch.
>>
>> Meh, I just noticed that the WARNING text claims "InitPlan" is the
>> function name. I think it's best to get rid of that. It's pretty much
>> redundant anyway if you do: \set VERBOSITY verbose
>
> Oops, good catch that one. Removed "InitPlan: " from the message in the
> attached.
>

I have looked at the patch (v9), and have no further comments. I can
confirm a speedup in the SELECT FOR SHARE case.

Thanks for working on this !

Best regards,
Jesper

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Andrew Dunstan 2018-09-28 13:51:51 Re: pgsql: Build src/port files as a library with -fPIC, and use that in li
Previous Message Tom Lane 2018-09-28 13:48:58 Re: pgsql: Build src/port files as a library with -fPIC, and use that in li