Hello Mike Percy, Adar Dembo,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/2513
to review the following change.
Change subject: Check for KUDU-418: we don't support changing TS hosts/ports
......................................................................
Check for KUDU-418: we don't support changing TS hosts/ports
This adds a check in the master that attempts to reject tablet
servers which have changed host/port. We don't currently support
this, so allowing them to register currently just ends up hitting
other more hard-to-diagnose issues later on.
This would have helped me troubleshoot some issues with client-test
where we were hitting this issue.
Change-Id: I16d9a91059b7dde63a2cac71e1fde0228b6a9f3d
---
M src/kudu/master/master-test.cc
M src/kudu/master/ts_descriptor.cc
2 files changed, 28 insertions(+), 0 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/13/2513/1
--
To view, visit http://gerrit.cloudera.org:8080/2513
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I16d9a91059b7dde63a2cac71e1fde0228b6a9f3d
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Mike Percy <[email protected]>
