On Mon, Feb 8, 2016 at 4:18 AM, Andres Freund <and...@anarazel.de> wrote: > I'm not really a fan. I'd rather change existing callers to add a > 'flags' bitmask argument. Right now there can't really be XLogInserts() > in extension code, so that's pretty ok to change.
Yeah, but to what benefit? You're just turning a smaller patch into a bigger one and requiring churning a bunch of code that wouldn't otherwise need to be touched. I think Michael has a good point. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (firstname.lastname@example.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers