[kudu-CR] Rename Remote Bootstrap: some manual changes
Todd Lipcon has submitted this change and it was merged. Change subject: Rename Remote Bootstrap: some manual changes .. Rename Remote Bootstrap: some manual changes This patch has a bunch more manual fixes to replace "Remote Bootstrap" terminology with "Tablet Copy". I looked through the previous (automatic) diff and tried to clean up comments and variable names to be more clear, as well as find other abbreviated cases like the term 'bootstrap' or 'rb' which needed to be updated. This still doesn't do the file renames. That's yet to come. Change-Id: I9e2860b64e46c5cc021cb8574e6845d7bcb27cce Reviewed-on: http://gerrit.cloudera.org:8080/3852 Reviewed-by: Mike Percy Tested-by: Kudu Jenkins --- M docs/design-docs/README.md M docs/design-docs/raft-config-change.md M docs/design-docs/raft-remote-bootstrap.md M docs/whitepaper/kudu.tex M src/kudu/consensus/consensus.proto M src/kudu/consensus/consensus_peers.cc M src/kudu/consensus/consensus_peers.h M src/kudu/consensus/consensus_queue-test.cc M src/kudu/consensus/consensus_queue.cc M src/kudu/consensus/consensus_queue.h M src/kudu/integration-tests/cluster_itest_util.cc M src/kudu/integration-tests/cluster_itest_util.h M src/kudu/integration-tests/delete_table-test.cc M src/kudu/integration-tests/remote_bootstrap-itest.cc M src/kudu/integration-tests/tablet_replacement-itest.cc M src/kudu/tablet/metadata.proto M src/kudu/tablet/tablet_bootstrap.cc M src/kudu/tserver/remote_bootstrap_client.cc M src/kudu/tserver/remote_bootstrap_client.h M src/kudu/tserver/remote_bootstrap_service.cc M src/kudu/tserver/ts_tablet_manager.cc 21 files changed, 167 insertions(+), 167 deletions(-) Approvals: Mike Percy: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/3852 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9e2860b64e46c5cc021cb8574e6845d7bcb27cce Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy Gerrit-Reviewer: Todd Lipcon
[kudu-CR] Rename remote bootstrap files to 'tablet copy'
Todd Lipcon has submitted this change and it was merged. Change subject: Rename remote bootstrap files to 'tablet copy' .. Rename remote bootstrap files to 'tablet copy' Change-Id: I3f31fb72d11d16ee05bf90be3a7c77a82e5db6f7 Reviewed-on: http://gerrit.cloudera.org:8080/3853 Tested-by: Kudu Jenkins Reviewed-by: Mike Percy --- M src/kudu/integration-tests/CMakeLists.txt R src/kudu/integration-tests/tablet_copy-itest.cc M src/kudu/tserver/CMakeLists.txt R src/kudu/tserver/tablet_copy-test-base.h R src/kudu/tserver/tablet_copy.proto R src/kudu/tserver/tablet_copy_client-test.cc R src/kudu/tserver/tablet_copy_client.cc R src/kudu/tserver/tablet_copy_client.h R src/kudu/tserver/tablet_copy_service-test.cc R src/kudu/tserver/tablet_copy_service.cc R src/kudu/tserver/tablet_copy_service.h R src/kudu/tserver/tablet_copy_session-test.cc R src/kudu/tserver/tablet_copy_session.cc R src/kudu/tserver/tablet_copy_session.h M src/kudu/tserver/tablet_server-test-base.h M src/kudu/tserver/tablet_server.cc M src/kudu/tserver/tablet_service.cc M src/kudu/tserver/ts_tablet_manager.cc 18 files changed, 31 insertions(+), 31 deletions(-) Approvals: Mike Percy: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/3853 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3f31fb72d11d16ee05bf90be3a7c77a82e5db6f7 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy Gerrit-Reviewer: Todd Lipcon
[kudu-CR] KUDU-1552: schema corruption cause kudu crash
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1552: schema corruption cause kudu crash .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/2733/ -- To view, visit http://gerrit.cloudera.org:8080/3856 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia66a39c0f1273c310e1fe19f52672e23f9771c41 Gerrit-PatchSet: 4 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: song bruce zhang Gerrit-Reviewer: Kudu Jenkins Gerrit-HasComments: No
[kudu-CR] KUDU-1552: schema corruption cause kudu crash
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3856 to look at the new patch set (#4). Change subject: KUDU-1552: schema corruption cause kudu crash .. KUDU-1552: schema corruption cause kudu crash In some corner case, schema does not have column ids, kudu-tserver will crash when starting up, after previous node crash. I placed a branch before check/dcheck and return Status::corruption if schema is corrupted. Change-Id: Ia66a39c0f1273c310e1fe19f52672e23f9771c41 --- M src/kudu/tablet/tablet.cc 1 file changed, 3 insertions(+), 1 deletion(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/56/3856/4 -- To view, visit http://gerrit.cloudera.org:8080/3856 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ia66a39c0f1273c310e1fe19f52672e23f9771c41 Gerrit-PatchSet: 4 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: song bruce zhang Gerrit-Reviewer: Kudu Jenkins
[kudu-CR] KUDU-1552: schema corruption cause kudu crash
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3856 to look at the new patch set (#3). Change subject: KUDU-1552: schema corruption cause kudu crash .. KUDU-1552: schema corruption cause kudu crash In some corner case, schema does not have column ids, kudu-tserver will crash when starting up, after previous node crash. I placed a branch before check/dcheck and return Status::corruption if schema is corrupted. Change-Id: Ia66a39c0f1273c310e1fe19f52672e23f9771c41 --- M src/kudu/tablet/tablet.cc 1 file changed, 3 insertions(+), 1 deletion(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/56/3856/3 -- To view, visit http://gerrit.cloudera.org:8080/3856 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ia66a39c0f1273c310e1fe19f52672e23f9771c41 Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: song bruce zhang Gerrit-Reviewer: Kudu Jenkins
[kudu-CR] KUDU-1552: schema corruption cause kudu crash
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1552: schema corruption cause kudu crash .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2732/ -- To view, visit http://gerrit.cloudera.org:8080/3856 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia66a39c0f1273c310e1fe19f52672e23f9771c41 Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: song bruce zhang Gerrit-Reviewer: Kudu Jenkins Gerrit-HasComments: No
[kudu-CR] KUDU-1552: schema corruption cause kudu crash
song bruce zhang has uploaded a new patch set (#2). Change subject: KUDU-1552: schema corruption cause kudu crash .. KUDU-1552: schema corruption cause kudu crash In some corner case, schema does not have column ids, kudu-tserver will crash when starting up, after previous node crash. I placed a branch before check/dcheck and return Status::corruption if schema is corrupted. Change-Id: Ia66a39c0f1273c310e1fe19f52672e23f9771c41 --- M src/kudu/tablet/tablet.cc 1 file changed, 3 insertions(+), 1 deletion(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/56/3856/2 -- To view, visit http://gerrit.cloudera.org:8080/3856 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ia66a39c0f1273c310e1fe19f52672e23f9771c41 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: song bruce zhang Gerrit-Reviewer: Kudu Jenkins
[kudu-CR] KUDU-1552: schema corruption cause kudu crash
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1552: schema corruption cause kudu crash .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2731/ -- To view, visit http://gerrit.cloudera.org:8080/3856 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia66a39c0f1273c310e1fe19f52672e23f9771c41 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: song bruce zhang Gerrit-Reviewer: Kudu Jenkins Gerrit-HasComments: No
[kudu-CR] KUDU-1552: schema corruption cause kudu crash
song bruce zhang has uploaded a new change for review. http://gerrit.cloudera.org:8080/3856 Change subject: KUDU-1552: schema corruption cause kudu crash .. KUDU-1552: schema corruption cause kudu crash In some corner case, schema does not have column ids, kudu-tserver will crash when starting up, after previous node crash. Change-Id: Ia66a39c0f1273c310e1fe19f52672e23f9771c41 --- M src/kudu/tablet/tablet.cc 1 file changed, 3 insertions(+), 1 deletion(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/56/3856/1 -- To view, visit http://gerrit.cloudera.org:8080/3856 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ia66a39c0f1273c310e1fe19f52672e23f9771c41 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: song bruce zhang
[kudu-CR] KUDU-1552: schema corruption cause kudu crash
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1552: schema corruption cause kudu crash .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2730/ -- To view, visit http://gerrit.cloudera.org:8080/3856 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia66a39c0f1273c310e1fe19f52672e23f9771c41 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: song bruce zhang Gerrit-Reviewer: Kudu Jenkins Gerrit-HasComments: No