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/08/07 19:01:51 UTC
[1/2] kudu git commit: Clean up spurious declarations in messenger.cc
Repository: kudu
Updated Branches:
refs/heads/master 943b1ae26 -> 1678b3dc9
Clean up spurious declarations in messenger.cc
Change-Id: I105524bab85bef9c08205e3d1f262fa5234745d9
Reviewed-on: http://gerrit.cloudera.org:8080/7583
Tested-by: Kudu Jenkins
Reviewed-by: Todd Lipcon <to...@apache.org>
Project: http://git-wip-us.apache.org/repos/asf/kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/6bdbd26b
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/6bdbd26b
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/6bdbd26b
Branch: refs/heads/master
Commit: 6bdbd26bc7d0c680f81ac38a05a65a080c63af55
Parents: 943b1ae
Author: Sailesh Mukil <sa...@apache.org>
Authored: Thu Aug 3 22:13:15 2017 -0700
Committer: Todd Lipcon <to...@apache.org>
Committed: Sat Aug 5 01:29:35 2017 +0000
----------------------------------------------------------------------
src/kudu/rpc/messenger.cc | 3 ---
1 file changed, 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/kudu/blob/6bdbd26b/src/kudu/rpc/messenger.cc
----------------------------------------------------------------------
diff --git a/src/kudu/rpc/messenger.cc b/src/kudu/rpc/messenger.cc
index 76b04fe..35fda52 100644
--- a/src/kudu/rpc/messenger.cc
+++ b/src/kudu/rpc/messenger.cc
@@ -119,9 +119,6 @@ DECLARE_bool(allow_world_readable_credentials);
namespace kudu {
namespace rpc {
-class Messenger;
-class ServerBuilder;
-
template <typename T>
static Status ParseTriState(const char* flag_name, const string& flag_value, T* tri_state) {
if (boost::iequals(flag_value, "required")) {
[2/2] kudu git commit: [tests] fix compilation warning on OS X
Posted by al...@apache.org.
[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 <ha...@cloudera.com>
Tested-by: Kudu Jenkins
Reviewed-by: David Ribeiro Alves <da...@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 <as...@cloudera.com>
Authored: Fri Aug 4 15:45:17 2017 -0700
Committer: Alexey Serbin <as...@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(