kudu-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ale...@apache.org
Subject [2/2] kudu git commit: [tests] fix compilation warning on OS X
Date Mon, 07 Aug 2017 19:01:52 GMT
[tests] fix compilation warning on OS X

Fixed compilation warning on OS X due to unused function:
CountBlocksUnderManagement().

This changelist does not contain any functional modifications.

Change-Id: I836453fd986589e53fcceaa19599eba615074e53
Reviewed-on: http://gerrit.cloudera.org:8080/7594
Reviewed-by: Hao Hao <hao.hao@cloudera.com>
Tested-by: Kudu Jenkins
Reviewed-by: David Ribeiro Alves <davidralves@gmail.com>


Project: http://git-wip-us.apache.org/repos/asf/kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/1678b3dc
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/1678b3dc
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/1678b3dc

Branch: refs/heads/master
Commit: 1678b3dc9b5bc81f57ebac03f2893835744ff098
Parents: 6bdbd26
Author: Alexey Serbin <aserbin@cloudera.com>
Authored: Fri Aug 4 15:45:17 2017 -0700
Committer: Alexey Serbin <aserbin@cloudera.com>
Committed: Mon Aug 7 18:59:33 2017 +0000

----------------------------------------------------------------------
 src/kudu/integration-tests/tablet_copy-itest.cc | 5 +++++
 1 file changed, 5 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/1678b3dc/src/kudu/integration-tests/tablet_copy-itest.cc
----------------------------------------------------------------------
diff --git a/src/kudu/integration-tests/tablet_copy-itest.cc b/src/kudu/integration-tests/tablet_copy-itest.cc
index ad10e5c..83c61d0 100644
--- a/src/kudu/integration-tests/tablet_copy-itest.cc
+++ b/src/kudu/integration-tests/tablet_copy-itest.cc
@@ -636,6 +636,9 @@ TEST_F(TabletCopyITest, TestDeleteLeaderDuringTabletCopyStressTest) {
 }
 
 namespace {
+#ifndef __APPLE__
+// CountBlocksUnderManagement() is used by routines which work with the
+// LogBlockManager (not used on OS X).
 int64_t CountBlocksUnderManagement(ExternalTabletServer* ets) {
   int64_t ret;
   CHECK_OK(ets->GetInt64Metric(
@@ -646,6 +649,8 @@ int64_t CountBlocksUnderManagement(ExternalTabletServer* ets) {
                &ret));
   return ret;
 }
+#endif // #ifndef __APPLE__
+
 int64_t CountUpdateConsensusCalls(ExternalTabletServer* ets) {
   int64_t ret;
   CHECK_OK(ets->GetInt64Metric(


Mime
View raw message