Re: speeding up planning with partitions

From: David Rowley <david(dot)rowley(at)2ndquadrant(dot)com>
To: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
Cc: Amit Langote <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp>, Amit Langote <amitlangote09(at)gmail(dot)com>, "Imai, Yoshikazu" <imai(dot)yoshikazu(at)jp(dot)fujitsu(dot)com>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: speeding up planning with partitions
Date: 2019-01-10 21:47:55
Message-ID: CAKJS1f988rsyhwvLgfT-y1UCYUfXDOv67ENQk=v24OxhsZOzZw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, 11 Jan 2019 at 06:56, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> wrote:
> Pushed 0001 with some minor tweaks, thanks.

Thanks for pushing. I had looked at 0001 last night and there wasn't
much to report, just:

v12 0001

1. I see you've moved translate_col_privs() out of prepunion.c into
appendinfo.c. This required making it an external function, but it's
only use is in inherit.c, so would it not be better to put it there
and keep it static?

2. The following two lines I think need to swap their order.

+#include "utils/rel.h"
+#include "utils/lsyscache.h"

Both are pretty minor details but thought I'd post them anyway.

--
David Rowley http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2019-01-10 22:03:30 Re: Improve selectivity estimate for range queries
Previous Message Fabien COELHO 2019-01-10 21:23:34 Re: [HACKERS] pgbench - allow to store select results into variables