On Thu, Oct 16, 2014 at 11:28 AM, Stephen Frost <sfr...@snowman.net> wrote: > On Thursday, October 16, 2014, Robert Haas <robertmh...@gmail.com> wrote: >> >> On Thu, Oct 16, 2014 at 9:49 AM, Stephen Frost <sfr...@snowman.net> 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 (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
