Dan Burkert has posted comments on this change.

Change subject: rpc: call-level feature flags
......................................................................


Patch Set 1:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/2239/1//COMMIT_MSG
Commit Message:

Line 18: Change-Id: I46e46bcf80b93fc6cce50f37dd71a6e6539047f8
> It's not clear how this connects with the other change.  I thought reading 
Done


http://gerrit.cloudera.org:8080/#/c/2239/1/src/kudu/rpc/rpc_header.proto
File src/kudu/rpc/rpc_header.proto:

Line 145:   // RPC_REQUIRED_FEATURE_FLAGS flag.
> In what situations is this not supported?  Is this only for servers built b
Yes, it's only for servers built before this patch.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I46e46bcf80b93fc6cce50f37dd71a6e6539047f8
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Jean-Daniel Cryans
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: Yes

Reply via email to