Skip site navigation (1) Skip section navigation (2)

Re: [PATCH 1/2] Provide a common malloc wrappers and palloc et al. emulation for frontend'ish environs

From: Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
To: Magnus Hagander <magnus(at)hagander(dot)net>
Cc: Andres Freund <andres(at)2ndquadrant(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [PATCH 1/2] Provide a common malloc wrappers and palloc et al. emulation for frontend'ish environs
Date: 2013-01-09 13:23:25
Message-ID: CAB7nPqRbcgx3LKVjnVQ5U1mVUyjSwXz5tVjzyCpTK5qq9hr2cw@mail.gmail.com (view raw or flat)
Thread:
Lists: pgsql-hackers
On Wed, Jan 9, 2013 at 9:54 PM, Magnus Hagander <magnus(at)hagander(dot)net> wrote:

> On Wed, Jan 9, 2013 at 1:47 PM, Andres Freund <andres(at)2ndquadrant(dot)com>
> wrote:
>
Yeah, so far it's also just my opinion in the other direction :)
> Hopefully, some others will have thoughts about it too.
>
Just giving my 2c here...

Instead of posting multiple 5~7 patches at the same time, why not limiting
the number of patches published at the same time to a lower number (max
2~3)? The logical replication implementation can be surely broken down into
many more pieces that could be reviewed carefully one by one, and in a way
that would make the implementation steps clearer than it is now for all the
people of this ML.
OK this would make the review process longer but the good point is that
some hackers who are only specialized in some areas of the PG code would be
able to give precious feedback.
-- 
Michael Paquier
http://michael.otacoo.com

In response to

Responses

pgsql-hackers by date

Next:From: Amit KapilaDate: 2013-01-09 13:45:52
Subject: Re: Extra XLOG in Checkpoint for StandbySnapshot
Previous:From: Peter EisentrautDate: 2013-01-09 13:17:52
Subject: askpass program for libpq

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group