[kudu-CR] KUDU-1353: remove per-tablet replica locations cache
Mike Percy has submitted this change and it was merged. 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 Reviewed-on: http://gerrit.cloudera.org:8080/2887 Tested-by: Adar Dembo Reviewed-by: Jean-Daniel Cryans Reviewed-by: Mike Percy --- 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(-) Approvals: Jean-Daniel Cryans: Looks good to me, but someone else must approve Mike Percy: Looks good to me, approved Adar Dembo: Verified -- To view, visit http://gerrit.cloudera.org:8080/2887 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6376b5307f75f5d505b33a5ff4262da619cd1d8d Gerrit-PatchSet: 12 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Mike Percy Gerrit-Reviewer: Todd Lipcon
[kudu-CR] KUDU-1353: remove per-tablet replica locations cache
Mike Percy has posted comments on this change. Change subject: KUDU-1353: remove per-tablet replica locations cache .. Patch Set 11: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/2887 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6376b5307f75f5d505b33a5ff4262da619cd1d8d Gerrit-PatchSet: 11 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Mike Percy Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: No
[kudu-CR] KUDU-1353: remove per-tablet replica locations cache
Adar Dembo has posted comments on this change. Change subject: KUDU-1353: remove per-tablet replica locations cache .. Patch Set 11: Another testDisconnect Java failure. -- To view, visit http://gerrit.cloudera.org:8080/2887 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6376b5307f75f5d505b33a5ff4262da619cd1d8d Gerrit-PatchSet: 11 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: No
[kudu-CR] KUDU-1353: remove per-tablet replica locations cache
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1353: remove per-tablet replica locations cache .. Patch Set 11: Build Started http://104.196.14.100/job/kudu-gerrit/1783/ -- To view, visit http://gerrit.cloudera.org:8080/2887 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6376b5307f75f5d505b33a5ff4262da619cd1d8d Gerrit-PatchSet: 11 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: No
[kudu-CR] KUDU-1353: remove per-tablet replica locations cache
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 Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Todd Lipcon
[kudu-CR] KUDU-1353: remove per-tablet replica locations cache
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1353: remove per-tablet replica locations cache .. Patch Set 10: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/2887 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6376b5307f75f5d505b33a5ff4262da619cd1d8d Gerrit-PatchSet: 10 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: No
[kudu-CR] KUDU-1353: remove per-tablet replica locations cache
Adar Dembo has posted comments on this change. Change subject: KUDU-1353: remove per-tablet replica locations cache .. Patch Set 10: Verified+1 Overriding Jenkins, the one failure was a Java test (testDisconnect) in TSAN mode. Dan has been working on that one. -- To view, visit http://gerrit.cloudera.org:8080/2887 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6376b5307f75f5d505b33a5ff4262da619cd1d8d Gerrit-PatchSet: 10 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: No
[kudu-CR] KUDU-1353: remove per-tablet replica locations cache
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1353: remove per-tablet replica locations cache .. Patch Set 10: Build Started http://104.196.14.100/job/kudu-gerrit/1778/ -- To view, visit http://gerrit.cloudera.org:8080/2887 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6376b5307f75f5d505b33a5ff4262da619cd1d8d Gerrit-PatchSet: 10 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: No
[kudu-CR] KUDU-1353: remove per-tablet replica locations cache
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/2887 to look at the new patch set (#10). 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/10 -- 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: 10 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon
[kudu-CR] KUDU-1353: remove per-tablet replica locations cache
Adar Dembo has posted comments on this change. Change subject: KUDU-1353: remove per-tablet replica locations cache .. Patch Set 9: (1 comment) http://gerrit.cloudera.org:8080/#/c/2887/9/src/kudu/master/master-path-handlers.cc File src/kudu/master/master-path-handlers.cc: Line 27: #include > is this supposed to be in brackets? Whoops, not sure why I did that. Fixed. -- To view, visit http://gerrit.cloudera.org:8080/2887 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6376b5307f75f5d505b33a5ff4262da619cd1d8d Gerrit-PatchSet: 9 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: Yes
[kudu-CR] KUDU-1353: remove per-tablet replica locations cache
Dan Burkert has posted comments on this change. Change subject: KUDU-1353: remove per-tablet replica locations cache .. Patch Set 9: (1 comment) http://gerrit.cloudera.org:8080/#/c/2887/9/src/kudu/master/master-path-handlers.cc File src/kudu/master/master-path-handlers.cc: Line 27: #include is this supposed to be in brackets? -- To view, visit http://gerrit.cloudera.org:8080/2887 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6376b5307f75f5d505b33a5ff4262da619cd1d8d Gerrit-PatchSet: 9 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: Yes
[kudu-CR] KUDU-1353: remove per-tablet replica locations cache
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1353: remove per-tablet replica locations cache .. Patch Set 9: Build Started http://104.196.14.100/job/kudu-gerrit/1760/ -- To view, visit http://gerrit.cloudera.org:8080/2887 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6376b5307f75f5d505b33a5ff4262da619cd1d8d Gerrit-PatchSet: 9 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: No
[kudu-CR] KUDU-1353: remove per-tablet replica locations cache
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/2887 to look at the new patch set (#9). 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/9 -- 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: 9 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon
[kudu-CR] KUDU-1353: remove per-tablet replica locations cache
Adar Dembo has posted comments on this change. Change subject: KUDU-1353: remove per-tablet replica locations cache .. Patch Set 8: (2 comments) http://gerrit.cloudera.org:8080/#/c/2887/8/src/kudu/master/catalog_manager.cc File src/kudu/master/catalog_manager.cc: Line 2516: LOG(WARNING) << "Not sending DeleteTablet requests; no consensus state for tablet " > is this warning-worthy? do you expect this to be common? No, I don't expect it to be common. Don't care strongly about the warning, will downgrade. Line 3011: locs_pb->set_deprecated_stale(false); > is this necessary? if the protobuf didn't specify a default, I think it's s But it's 'required' in the protobuf and has no default value, so doesn't that mean we must set it always and forever? -- To view, visit http://gerrit.cloudera.org:8080/2887 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6376b5307f75f5d505b33a5ff4262da619cd1d8d Gerrit-PatchSet: 8 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: Yes
[kudu-CR] KUDU-1353: remove per-tablet replica locations cache
Todd Lipcon has posted comments on this change. Change subject: KUDU-1353: remove per-tablet replica locations cache .. Patch Set 8: (2 comments) http://gerrit.cloudera.org:8080/#/c/2887/8/src/kudu/master/catalog_manager.cc File src/kudu/master/catalog_manager.cc: Line 2516: LOG(WARNING) << "Not sending DeleteTablet requests; no consensus state for tablet " is this warning-worthy? do you expect this to be common? Line 3011: locs_pb->set_deprecated_stale(false); is this necessary? if the protobuf didn't specify a default, I think it's safe to not set it (and old clients will read false) -- To view, visit http://gerrit.cloudera.org:8080/2887 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6376b5307f75f5d505b33a5ff4262da619cd1d8d Gerrit-PatchSet: 8 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: Yes
[kudu-CR] KUDU-1353: remove per-tablet replica locations cache
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1353: remove per-tablet replica locations cache .. Patch Set 8: Build Started http://104.196.14.100/job/kudu-gerrit/1752/ -- To view, visit http://gerrit.cloudera.org:8080/2887 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6376b5307f75f5d505b33a5ff4262da619cd1d8d Gerrit-PatchSet: 8 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: No
[kudu-CR] KUDU-1353: remove per-tablet replica locations cache
Adar Dembo has posted comments on this change. Change subject: KUDU-1353: remove per-tablet replica locations cache .. Patch Set 7: I've rebased this on top of a new patch that should fix the test failures I mentioned earlier. -- To view, visit http://gerrit.cloudera.org:8080/2887 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6376b5307f75f5d505b33a5ff4262da619cd1d8d Gerrit-PatchSet: 7 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: No
[kudu-CR] KUDU-1353: remove per-tablet replica locations cache
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1353: remove per-tablet replica locations cache .. Patch Set 7: Build Started http://104.196.14.100/job/kudu-gerrit/1743/ -- To view, visit http://gerrit.cloudera.org:8080/2887 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6376b5307f75f5d505b33a5ff4262da619cd1d8d Gerrit-PatchSet: 7 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: No
[kudu-CR] KUDU-1353: remove per-tablet replica locations cache
Hello Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/2887 to look at the new patch set (#7). 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/7 -- 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: 7 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon