On Jul 2, 2012, at 12:04 AM, Robert Haas <robertmh...@gmail.com> wrote: > On Jul 1, 2012, at 4:18 PM, Tom Lane <t...@sss.pgh.pa.us> wrote: >> However, I'm a bit worried by the "if (!FirstSnapshotSet)" restriction >> in GetLatestSnapshot. Are we sure that enum comparisons could never >> happen without a snapshot already being set? What's the point of >> throwing an error there anyway, as opposed to letting it redirect to >> GetTransactionSnapshot? > > I don't know whether it should set the transaction snapshot or just r
Argh, sorry. ...or just return a current snapshot, and I also don't know whether it needs to be changed because of this; but I agree with changing it. Erroring out always seemed kind of pointless to me... ...Robert -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
