On Fri, Nov 22, 2013 at 04:38:53PM -0800, Kevin Grittner wrote: > Andres Freund <[email protected]> wrote: > > On 2013-11-22 13:34:18 -0800, Kevin Grittner wrote: > >> Oddly, it didn't complain about creating users within a read-only > >> transaction. That seems like a potential bug. > > > > There's lots of things that escape XactReadOnly. I've thought (and I > > think suggested) before that we should put in another layer of defense > > by also putting a check in AssignTransactionId(). Imo the compatibility > > woes (like not being able to run SELECT txid_current();) are well worth > > the nearly ironclad guarantee that we're not writing. > > I agree that something like that is would be a good idea; however, > I'm sure you would agree that would not be material for a > back-patch to a stable branch.
I am not a fan of backpatching any of this. We have learned the fix is more complex than thought, and the risk of breakage and having pg_dump diffs change between minor releases doesn't seem justified. -- Bruce Momjian <[email protected]> http://momjian.us EnterpriseDB http://enterprisedb.com + Everyone has their own god. + -- Sent via pgsql-hackers mailing list ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
