Hello Alexey Serbin, Dan Burkert,

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

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

to review the following change.


Change subject: mini-cluster: condition on NO_TESTS=0
......................................................................

mini-cluster: condition on NO_TESTS=0

This module will be incorporated into the kudu CLI in the near future, but
for now this patch fixes builds that set NO_TESTS=1.

Our precommit builds don't disable tests, so I tested this by running a
local build with NO_TESTS=1. Without this patch, the build failed while
linking libmini-cluster.so. With it, the build passes.

Change-Id: I195b3ed5ed50dfbe7afb3004182bf02a3bcda31a
---
M src/kudu/mini-cluster/CMakeLists.txt
1 file changed, 5 insertions(+), 0 deletions(-)



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I195b3ed5ed50dfbe7afb3004182bf02a3bcda31a
Gerrit-Change-Number: 8199
Gerrit-PatchSet: 1
Gerrit-Owner: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>

Reply via email to