Hello David Ribeiro Alves, Alexey Serbin, I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/7904 to review the following change. Change subject: tablet copy: Allow voting from failed initial tablet copies ...................................................................... tablet copy: Allow voting from failed initial tablet copies This patch changes the behavior of tablets being copied for the first time, i.e. to a new tablet server that has never hosted that replica. In that case, if the tablet copy fails, with this patch that replica will still be allowed to vote while tombstoned. This improves availability. Without this patch, If a tablet copy to a new replica fails, that replica will not be able to vote while tombstoned because its last-logged opid field in its superblock will be unset. This is achieved within tablet copy by setting the initial tombstoned_last_logged_opid to (1,0). This patch is a bit of a hack, since it takes advantage of the fact that it is impossible to have a "real" OpId of 1.0 in the current implementation (1.1 is the operational minimum) so having a last-logged opid of 1.0 represents having never logged any actual ops. Change-Id: Ibaad2f56797db731a0f701bbd37e3e450bc17b51 --- M src/kudu/integration-tests/tablet_copy-itest.cc M src/kudu/tserver/tablet_copy_client.cc 2 files changed, 84 insertions(+), 0 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/04/7904/1 -- To view, visit http://gerrit.cloudera.org:8080/7904 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ibaad2f56797db731a0f701bbd37e3e450bc17b51 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: David Ribeiro Alves <davidral...@gmail.com>