Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11593 )
Change subject: Move constructor and assignment operator for Schema ...................................................................... Move constructor and assignment operator for Schema This adds a move constructor and move assignment operator to Schema. I removed Schema::swap. It was only used in tests, and any potential use of it should be adequately fulfilled by one of moves, copies, or Schema::Reset. Change-Id: I6c827c84657875be4cf2bc565db03a686849d8e2 Reviewed-on: http://gerrit.cloudera.org:8080/11593 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Kudu Jenkins Reviewed-by: Alexey Serbin <[email protected]> --- M src/kudu/common/id_mapping.h M src/kudu/common/schema-test.cc M src/kudu/common/schema.cc M src/kudu/common/schema.h 4 files changed, 115 insertions(+), 56 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/11593 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I6c827c84657875be4cf2bc565db03a686849d8e2 Gerrit-Change-Number: 11593 Gerrit-PatchSet: 7 Gerrit-Owner: Will Berkeley <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Will Berkeley <[email protected]>
