Mike Percy has uploaded a new change for review.

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

Change subject: Revert "KUDU-1490: Show software version of tablet servers"
......................................................................

Revert "KUDU-1490: Show software version of tablet servers"

This reverts commit 1b7abe081817b5f9af979407df105a08dd226521.

After reverting this commit and applying the previous patch, downgrade
works.

The reason for the problem is that there is some validation in
TSDescriptor::Register() in ts_descriptor.cc on line 65
that returns an error if the ShortDebugString() of two TSDescriptor
objects must not change between registrations. Since this adds a field,
that check is triggered. If you downgrade, you get the following error
message over and over on the master:

E0819 01:56:10.421804 32374 ts_descriptor.cc:73] Tablet server 
e1b61aaede6844049378351a3eb42f73 is attempting to re-register with a different 
host/port. This is not currently supported.
Old: {rpc_addresses { host: "mpercy-ThinkPad-T530" port: 7050 } http_addresses 
{ host: "mpercy-ThinkPad-T530" port: 8050 } 3: "kudu 0.10.0 (rev 
4686cc4f11a03507f8eca5ee7cc1653d323dba72)"}
New: {rpc_addresses { host: "mpercy-ThinkPad-T530" port: 7050 } http_addresses 
{ host: "mpercy-ThinkPad-T530" port: 8050 }}

Change-Id: I8a873a01404ae04018511bd2899b8a510df71722
---
M src/kudu/integration-tests/registration-test.cc
M src/kudu/master/master.proto
M src/kudu/tserver/heartbeater.cc
3 files changed, 2 insertions(+), 9 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/60/4060/1
-- 
To view, visit http://gerrit.cloudera.org:8080/4060
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8a873a01404ae04018511bd2899b8a510df71722
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Mike Percy <mpe...@apache.org>

Reply via email to