On Thu, Oct 16, 2014 at 11:28 AM, Stephen Frost <[email protected]> wrote: > On Thursday, October 16, 2014, Robert Haas <[email protected]> wrote: >> >> On Thu, Oct 16, 2014 at 9:49 AM, Stephen Frost <[email protected]> wrote: >> > As a side-note, this change is included in the 'role attributes' patch. >> >> It's really important that we keep separate changes in separate >> patches that are committed in separate commits. Otherwise, it gets >> really confusing. > > I can do that, but it overlaps with the MONITORING role attribute changes > also..
I'm not sure what your point is. Whether keeping changes separate is easy or hard, and whether things overlap with multiple other things or just one, we need to make the effort to do it. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
