You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kudu.apache.org by "Todd Lipcon (Code Review)" <ge...@cloudera.org> on 2016/02/19 03:18:38 UTC

[kudu-CR] WIP: call-level feature flags

Hello Dan Burkert, Jean-Daniel Cryans,

I'd like you to do a code review.  Please visit

    http://gerrit.cloudera.org:8080/2239

to review the following change.

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

WIP: call-level feature flags

This adds a new API RpcController::RequireServerFeature

WIP:
- not implemented yet, just want feedback on client API
- not sure yet the best place to put the server API where
  the service impl lists which features it supports,
  or the code which checks and responds with an error
  when unsupported.

Change-Id: I46e46bcf80b93fc6cce50f37dd71a6e6539047f8
---
M src/kudu/rpc/rpc_controller.h
M src/kudu/rpc/rpc_header.proto
M src/kudu/rpc/rtest.proto
3 files changed, 65 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/39/2239/1
-- 
To view, visit http://gerrit.cloudera.org:8080/2239
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I46e46bcf80b93fc6cce50f37dd71a6e6539047f8
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Dan Burkert <da...@cloudera.com>
Gerrit-Reviewer: Jean-Daniel Cryans