Dan Ports <drkp@csail.mit.edu> wrote:
> Specifically, the problem is a missing check in
> OnConflict_CheckForSerializationFailure. We check whether the
> conflict has caused the writer to become a pivot, but only if
> neither the reader or writer is committed. Why is that last
> condition there? In this case, the reader (T4) has committed but
> the writer (T1) hasn't.
OK, I misread your post -- you are looking at T1 as the pivot, and
that test *is* the problem. When T1 becomes the pivot the reader
(T4) is committed, but it committed *after* T2. I can submit a
patch for that this evening, after testing to confirm that if finds
the T1 pivot, unless you want to get it.
Sorry for the misunderstanding. I'm sneaking peeks at this during
compiles of other stuff....
-Kevin
