You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by al...@apache.org on 2017/11/17 07:41:35 UTC

kudu git commit: Fix compiler warnings in tablet_replacement-itest

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 <as...@cloudera.com>
Tested-by: Alexey Serbin <as...@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 <mp...@apache.org>
Authored: Thu Nov 16 18:22:08 2017 -0800
Committer: Alexey Serbin <as...@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;
   }