Adar Dembo has posted comments on this change.

Change subject: rpc: bump default max message size to 50MB
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/2889/1/src/kudu/tserver/remote_bootstrap_session.cc
File src/kudu/tserver/remote_bootstrap_session.cc:

Line 172: CHECK_GT(requested_len, 0) << "rpc_max_message_size is too low to 
transfer data: "
        :                                      << FLAGS_rpc_max_message_size;
Seems odd to check this with each read and not once via gflag validator.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I1df399c5dd28ec9539fbd701d26ff2c7467991c8
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <[email protected]>
Gerrit-HasComments: Yes

Reply via email to