Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11763 )
Change subject: [master] update placement logic for RF % 2 == 0 ...................................................................... [master] update placement logic for RF % 2 == 0 Updated the logic of the replica placement in master to properly handle even replication factors. Added corresponding unit tests as well. It's possible to configure Kudu to allow creation of tables with an even replication factor. I think it's easier to implement the handling of those cases instead of handling possible inconsistencies in the rebalancer and other tools and adding extra paragraphs into release notes. Change-Id: I4259f805090dd350f31bf3bf2b6477898214ece4 Reviewed-on: http://gerrit.cloudera.org:8080/11763 Tested-by: Kudu Jenkins Reviewed-by: Will Berkeley <wdberke...@gmail.com> --- M src/kudu/master/placement_policy-test.cc M src/kudu/master/placement_policy.cc M src/kudu/master/placement_policy.h 3 files changed, 137 insertions(+), 5 deletions(-) Approvals: Kudu Jenkins: Verified Will Berkeley: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/11763 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I4259f805090dd350f31bf3bf2b6477898214ece4 Gerrit-Change-Number: 11763 Gerrit-PatchSet: 3 Gerrit-Owner: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com>
