You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kudu.apache.org by "Todd Lipcon (Code Review)" <ge...@cloudera.org> on 2016/03/15 02:04:31 UTC

[kudu-CR] KUDU-418 (part 1): enforce that we don't support changing TS hosts/ports

Hello Kudu Jenkins,

I'd like you to reexamine a change.  Please visit

    http://gerrit.cloudera.org:8080/2513

to look at the new patch set (#2).

Change subject: KUDU-418 (part 1): enforce that we don't support changing TS hosts/ports
......................................................................

KUDU-418 (part 1): enforce that 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 check will end up being rolled back when we actually implement a proper
fix for KUDU-418. In the meantime, this would have helped me troubleshoot some
issues with client-test where we hit 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/2
-- 
To view, visit http://gerrit.cloudera.org:8080/2513
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I16d9a91059b7dde63a2cac71e1fde0228b6a9f3d
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>