search

tikv / sig-transaction

Resources for the transaction SIG
62 stars 13 forks source link

Compatibility between async commit fallback and CDC #79

Closed sticnarf closed 3 years ago

sticnarf commented 3 years ago

If we fallback from async commit using the solution 2 in https://github.com/tikv/sig-transaction/issues/64, we need to amend the primary lock clearing the async commit mark. Then, there can be two mutations writing the same lock. I am not sure whether CDC can handle this case.

sticnarf commented 3 years ago

cc @5kbpers @MyonKeminta

youjiali1995 commented 3 years ago

We write the same lock when refreshes TTL too. I think CDC can handle it.

sticnarf commented 3 years ago

I write a new design of implementing fallback. It does not rewrite the primary lock (it's actually unnecessary, I think).