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

From Michael Paquier
Subject Re: [PATCH 1/2] Provide a common malloc wrappers and palloc et al. emulation for frontend'ish environs
Date
Msg-id CAB7nPqRbcgx3LKVjnVQ5U1mVUyjSwXz5tVjzyCpTK5qq9hr2cw@mail.gmail.com
Whole thread Raw
In response to Re: [PATCH 1/2] Provide a common malloc wrappers and palloc et al. emulation for frontend'ish environs  (Magnus Hagander <magnus@hagander.net>)
Responses Re: [PATCH 1/2] Provide a common malloc wrappers and palloc et al. emulation for frontend'ish environs
List pgsql-hackers


On Wed, Jan 9, 2013 at 9:54 PM, Magnus Hagander <magnus@hagander.net> wrote:
On Wed, Jan 9, 2013 at 1:47 PM, Andres Freund <andres@2ndquadrant.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

pgsql-hackers by date:

Previous
From: Peter Eisentraut
Date:
Subject: askpass program for libpq
Next
From: Amit Kapila
Date:
Subject: Re: Extra XLOG in Checkpoint for StandbySnapshot