David Ribeiro Alves has posted comments on this change.

Change subject: Allow tablet shutdown without completing txs
......................................................................


Patch Set 12:

Thinking a bit more which constraint we actually want to enforce. I think we 
need to enforce the following one:
If a transaction with timestamp t is being aborted post-apply, then no other 
transaction with a timestamp t1 > t can be made visible either through scans or 
tablet copy.

In practice I think this might not be too hard to enforce. When applying 
something on the tablet that gets an ioerror we could have it immediately 
change the state of the mvcc manager (not have the tablet manager do so). Note 
that I'm just referring to the mvcc manager and not to the replica, that can 
shutdown later.

We could also have a "shutdown timestamp", and then make any new scans and 
continuing scans make sure that their scan timestamp is lower than this one.

-- 
To view, visit http://gerrit.cloudera.org:8080/7439
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I983620f27e7226806a2cca253db7619731914d42
Gerrit-PatchSet: 12
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: David Ribeiro Alves <davidral...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mpe...@apache.org>
Gerrit-Reviewer: Tidy Bot
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-HasComments: No

Reply via email to