Code review comment for lp:~pbeaman/akiban-persistit/fix-accumulator-checkpoint-failure

Revision history for this message
Peter Beaman (pbeaman) wrote :

Agree about the magic value. Changed it so that updateBaseValue passes the actual commitTimestamp. Also changed break to return in +47. I think that's fine. The important thing is to ensure that _checkpointRef is always set when it needs to be, and I think that property is preserved.

« Back to merge proposal