On Sun, Dec 4, 2011 at 11:11 PM, Serge Dubrouski <[email protected]> wrote: > Florian, Dejan how would you like to merge a patch when we are ready? The > patch will be rather big one and AFAIK you have some policy on the amount of > changes for one patch.
If it's a big addition of functionality, then a big patch is expected. However please make sure that you do one patch per functional change. Also, don't mix functional changes with "cleanup" work like fixing whitespace, correcting incorrectly advertised resource parameters, etc. It's acceptable to mix those in with the same pull request, but they should be in separate changesets so we can easily bisect any arising issues. Other than that, I guess Dejan will agree with me that your PostgreSQL expertise is way better than his and mine. So if you greenlight the feature addition functionally we're unlikely to second-guess you on that. Does this help? Cheers, Florian _______________________________________________________ Linux-HA-Dev: [email protected] http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev Home Page: http://linux-ha.org/
