On Jul 2, 2012, at 12:04 AM, Robert Haas <[email protected]> wrote: > On Jul 1, 2012, at 4:18 PM, Tom Lane <[email protected]> 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 ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
