Hello Jean-Daniel Cryans, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/2887 to look at the new patch set (#11). Change subject: KUDU-1353: remove per-tablet replica locations cache ...................................................................... KUDU-1353: remove per-tablet replica locations cache This is the simpler alternative to rebuilding the caches on metadata load. Instead of using the caches to retrieve tserver RPC addresses, we'll look the addresses up on the spot using the TSDescriptor cache. The two caches should be equally coherent, as both were populated as a result of the master receiving TS heartbeats. I've also done away with the concept of a 'stale' locations response. The distinction can be real, but it's confusing, and more importantly, clients never took advantage of it. And if we're going to persist TSDescriptor data in master state as has been recently discussed, the distinction becomes even less useful. Change-Id: I6376b5307f75f5d505b33a5ff4262da619cd1d8d --- M java/kudu-client/src/main/java/org/kududb/client/GetMasterRegistrationReceived.java M java/kudu-client/src/test/java/org/kududb/client/TestAsyncKuduClient.java M src/kudu/client/client-test.cc M src/kudu/integration-tests/mini_cluster.cc M src/kudu/master/catalog_manager.cc M src/kudu/master/catalog_manager.h M src/kudu/master/master-path-handlers.cc M src/kudu/master/master-path-handlers.h M src/kudu/master/master-test-util.h M src/kudu/master/master.proto M src/kudu/master/ts_manager.cc M src/kudu/master/ts_manager.h M src/kudu/tools/create-demo-table.cc 13 files changed, 174 insertions(+), 306 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/87/2887/11 -- To view, visit http://gerrit.cloudera.org:8080/2887 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I6376b5307f75f5d505b33a5ff4262da619cd1d8d Gerrit-PatchSet: 11 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Dan Burkert <d...@cloudera.com> Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Todd Lipcon <t...@apache.org>