Re: [PATCH 1/2] Provide a common malloc wrappers and palloc et al. emulation for frontend'ish environs
| От | Michael Paquier |
|---|---|
| Тема | Re: [PATCH 1/2] Provide a common malloc wrappers and palloc et al. emulation for frontend'ish environs |
| Дата | |
| Msg-id | CAB7nPqRbcgx3LKVjnVQ5U1mVUyjSwXz5tVjzyCpTK5qq9hr2cw@mail.gmail.com обсуждение исходный текст |
| Ответ на | Re: [PATCH 1/2] Provide a common malloc wrappers and palloc et al. emulation for frontend'ish environs (Magnus Hagander <magnus@hagander.net>) |
| Ответы |
Re: [PATCH 1/2] Provide a common malloc wrappers and
palloc et al. emulation for frontend'ish environs
|
| Список | pgsql-hackers |
On Wed, Jan 9, 2013 at 9:54 PM, Magnus Hagander <magnus@hagander.net> wrote:
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.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.
--
Michael Paquier
http://michael.otacoo.com
В списке pgsql-hackers по дате отправления: