Hello Kudu Jenkins,

I'd like you to reexamine a change.  Please visit

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

to look at the new patch set (#2).

Change subject: [tools] introduced GetKuduCtlAbsolutePath()
......................................................................

[tools] introduced GetKuduCtlAbsolutePath()

This is a small clean-up on the Kudu command-line tools: the code
which returns the absolute path to the top-level kudu admin tool
is separated into GetKuduCtlAbsolutePath() utility method to avoid
code duplication.

Change-Id: Ib7109859fd4cbc6d380c358fd2c4dc3d00c6e874
---
M src/kudu/tools/CMakeLists.txt
M src/kudu/tools/kudu-admin-test.cc
M src/kudu/tools/kudu-tool-test.cc
M src/kudu/tools/kudu-ts-cli-test.cc
A src/kudu/tools/tool_test_util.cc
A src/kudu/tools/tool_test_util.h
6 files changed, 96 insertions(+), 46 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/47/4547/2
-- 
To view, visit http://gerrit.cloudera.org:8080/4547
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ib7109859fd4cbc6d380c358fd2c4dc3d00c6e874
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins

Reply via email to