kudu-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ale...@apache.org
Subject kudu git commit: Fix compiler warnings in tablet_replacement-itest
Date Fri, 17 Nov 2017 07:41:35 GMT
Repository: kudu
Updated Branches:
  refs/heads/master 338cf61cb -> 471feccd0


Fix compiler warnings in tablet_replacement-itest

This patch removes the following compiler warnings:

  [879/1023] Building CXX object src/kudu/inte...ment-itest.dir/tablet_replacement-itest.cc.o
  ../../src/kudu/integration-tests/tablet_replacement-itest.cc:370:7: warning: ignoring return
value of function declared with 'warn_unused_result' attribute [-Wunused-result]
        cluster_->tablet_server(kFollower1Index)->Pause();
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  ../../src/kudu/integration-tests/tablet_replacement-itest.cc:371:7: warning: ignoring return
value of function declared with 'warn_unused_result' attribute [-Wunused-result]
        cluster_->tablet_server(kFollower2Index)->Pause();
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Change-Id: I5f9f65a73d2fe0e4b6d75010628fd36a57f98def
Reviewed-on: http://gerrit.cloudera.org:8080/8584
Reviewed-by: Alexey Serbin <aserbin@cloudera.com>
Tested-by: Alexey Serbin <aserbin@cloudera.com>


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

Branch: refs/heads/master
Commit: 471feccd073e90ac767ce6d6339b4fda71f72ff0
Parents: 338cf61
Author: Mike Percy <mpercy@apache.org>
Authored: Thu Nov 16 18:22:08 2017 -0800
Committer: Alexey Serbin <aserbin@cloudera.com>
Committed: Fri Nov 17 07:40:36 2017 +0000

----------------------------------------------------------------------
 src/kudu/integration-tests/tablet_replacement-itest.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/471feccd/src/kudu/integration-tests/tablet_replacement-itest.cc
----------------------------------------------------------------------
diff --git a/src/kudu/integration-tests/tablet_replacement-itest.cc b/src/kudu/integration-tests/tablet_replacement-itest.cc
index c012f02..eb724f4 100644
--- a/src/kudu/integration-tests/tablet_replacement-itest.cc
+++ b/src/kudu/integration-tests/tablet_replacement-itest.cc
@@ -367,8 +367,8 @@ void TabletReplacementITest::TestDontEvictIfRemainingConfigIsUnstable(Instabilit
       cluster_->tablet_server(kFollower2Index)->Shutdown();
       break;
     case NODE_STOPPED:
-      cluster_->tablet_server(kFollower1Index)->Pause();
-      cluster_->tablet_server(kFollower2Index)->Pause();
+      ASSERT_OK(cluster_->tablet_server(kFollower1Index)->Pause());
+      ASSERT_OK(cluster_->tablet_server(kFollower2Index)->Pause());
       break;
   }
 


Mime
View raw message