Robert Haas wrote:
> On Mon, Dec 1, 2014 at 5:34 PM, Alvaro Herrera <alvhe...@2ndquadrant.com> 
> wrote:
> > I made two more changes:
> > 1. introduce newestCommitTs.  Original code was using lastCommitXact to
> > check that no "future" transaction is asked for, but this doesn't really
> > work if a long-running transaction is committed, because asking for
> > transactions with a higher Xid but which were committed earlier would
> > raise an error.
> 
> I'm kind of disappointed that, in spite of previous review comments,
> this got committed with extensive use of the CommitTs naming.  I think
> that's confusing, but it's also something that will be awkward if we
> want to add other data, such as the much-discussed commit LSN, to the
> facility.

I never saw a comment that CommitTs was an unwanted name.  There were
some that said that committs wasn't liked because it looked like a
misspelling, so we added an underscore -- stuff in lower case is
commit_ts everywhere.  Stuff in camel case didn't get the underscore
because it didn't seem necessary.  But other than that issue, the name
wasn't questioned, as far as I'm aware.

-- 
Álvaro Herrera                http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to