Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15013 )
Change subject: [server] validate RPC vs Raft maximum message size flags ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/15013/1/src/kudu/tserver/tablet_server.cc File src/kudu/tserver/tablet_server.cc: PS1: > Wouldn't you rather define this somewhere in consensus so that masters can Masters benefit from this as well since tserver is linked into master. But maybe consensus is a better place, especially if krpc library is linked into the consensus library. I'll give it a shot. -- To view, visit http://gerrit.cloudera.org:8080/15013 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ied3c2c7ee4f0df48636f5f7ed3adce1e69c28f9d Gerrit-Change-Number: 15013 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Tue, 14 Jan 2020 01:19:25 +0000 Gerrit-HasComments: Yes
