Yingchun Lai has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/19393 )

Change subject: [KUDU-3430] Implement getting flags filter logic in tool side
......................................................................

[KUDU-3430] Implement getting flags filter logic in tool side

When using new version of Kudu tool (version: 1.16.0) to
execute command: "kudu tserver get_flags localhost:7050
-flags=block_cache_capacity_mb" on the old version of Kudu
TServer (version: 1.10.1),it returned all flags not the
specified flags.

That because the filter logic was not implemented on the
server side (Kudu version 1.10.1).

So it is better to implement the filter logic on the Kudu
tool side also.

Change-Id: I0a2ddc2540d7aa70d3dd2e6c4101bb227e94c858
Reviewed-on: http://gerrit.cloudera.org:8080/19393
Reviewed-by: Yingchun Lai <[email protected]>
Tested-by: Yingchun Lai <[email protected]>
---
M src/kudu/server/generic_service.cc
M src/kudu/tools/kudu-tool-test.cc
M src/kudu/tools/tool_action_common.cc
3 files changed, 57 insertions(+), 15 deletions(-)

Approvals:
  Yingchun Lai: Looks good to me, approved; Verified

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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I0a2ddc2540d7aa70d3dd2e6c4101bb227e94c858
Gerrit-Change-Number: 19393
Gerrit-PatchSet: 12
Gerrit-Owner: Wang Xixu <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Wang Xixu <[email protected]>
Gerrit-Reviewer: Yingchun Lai <[email protected]>

Reply via email to