|From:||Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>|
|To:||Amit Langote <amitlangote09(at)gmail(dot)com>|
|Cc:||Simon Riggs <simon(at)2ndquadrant(dot)com>, Amit Langote <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp>, Robert Haas <robertmhaas(at)gmail(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>|
|Subject:||Re: Declarative partitioning|
|Views:||Raw Message | Whole Thread | Download mbox | Resend email|
Amit Langote wrote:
> On Fri, Nov 20, 2015 at 7:20 PM, Simon Riggs <simon(at)2ndquadrant(dot)com> wrote:
> > Drop it?? I think he means "in this initial patch", right Amit L ?
> Yes, there was some notion of multi-level partitioning in the earlier
> patch but I removed it from the version I posted on Oct 30. I do
> intend to re-introduce it though.
I'm with Simon. In my own experience, it's crazy to try to introduce a
huge feature such as this one in a single enormous commit. The last
patch you posted was 300 kb without any SGML changes.
The way parallel query is being introduced is a good example to follow
(or logical multi-master replication, for that matter) --- one
infrastructure piece at a time.
Let's build incrementally.
Álvaro Herrera http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
|Next Message||Ildus Kurbangaliev||2015-11-20 11:53:14||Re: [PATCH] Refactoring of LWLock tranches|
|Previous Message||Amit Langote||2015-11-20 11:09:38||Re: Declarative partitioning|