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