Adar Dembo has posted comments on this change.

Change subject: master: only use tablet reports to notify master of altered 
tablets
......................................................................


Patch Set 1:

> this change potentially changes the latency of alter table requests
 > right? this is likely not a bad thing , but you didn't need to
 > change timeouts on tests? did you loop this?

Agreed about the latency change, but I didn't loop it because I don't expect 
the timing to change significantly. Marking a tablet as dirty also means the 
tserver will send a heartbeat immediately, so we're talking about a few more ms 
for the heartbeat RPC.

-- 
To view, visit http://gerrit.cloudera.org:8080/3580
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I09ff1560d0bd5cecefb08684ecf5c2c992d69d46
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: David Ribeiro Alves <dral...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-HasComments: No

Reply via email to