From: | Beena Emerson <memissemerson(at)gmail(dot)com> |
---|---|
To: | David Rowley <david(dot)rowley(at)2ndquadrant(dot)com> |
Cc: | Amit Langote <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp>, Robert Haas <robertmhaas(at)gmail(dot)com>, amul sul <sulamul(at)gmail(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>, Dilip Kumar <dilipbalaut(at)gmail(dot)com> |
Subject: | Re: [HACKERS] Runtime Partition Pruning |
Date: | 2017-12-21 13:06:04 |
Message-ID: | CAOG9ApF_65Ogd4x02i3DaatsOpJcjJzcbpwcZt6AT4+VMnPDXQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hello,
On Thu, Dec 21, 2017 at 6:30 PM, David Rowley
<david(dot)rowley(at)2ndquadrant(dot)com> wrote:
> On 22 December 2017 at 01:43, Beena Emerson <memissemerson(at)gmail(dot)com> wrote:
>>> I wonder if Robert understands the extent of the problem. The patch
>>> will be useful to prune away partitions when the partitioned table is
>>> on the inside of a parameterised nested loop join, but I think we'll
>>> likely get some complaints about PREPARE statements always using a
>>> custom plan. It's going to make this pretty hard to write regression
>>> tests for unless we went and invent some GUC "prefer_generic_plan", or
>>> "generic_plan_cost_multiplier" to provide a way to coax
>>> choose_custom_plan() into not choosing a custom plan. I admit to not
>>> having the answer to this but I don't think we should rule out
>>> discussing possible fixes.
>>
>> I have attached the regression tests I am currently using to check my
>> patch. It has few prepared statements using runtime pruning.
>
> Thanks for sending. Will the tests still pass if you remove the hack
> from choose_custom_plan()?
>
The hack has been removed in the v7 patch sent and yes the tests work
without any additional adjustments.
Thank you,
Beena Emerson
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2017-12-21 13:11:07 | Re: Fix permissions check on pg_stat_get_wal_senders |
Previous Message | Ildar Musin | 2017-12-21 13:02:06 | Re: General purpose hashing func in pgbench |