diff --git a/BUILD b/BUILD index 64d1faf..bfb89c2 100644 --- a/BUILD +++ b/BUILD @@ -33,7 +33,7 @@ cc_library( "-Woverloaded-virtual", "-Wnon-virtual-dtor", "-Wno-missing-field-initializers", - "-std=c++11", + "-std=c++17", "-DGFLAGS_NS=google", ], linkopts = [ diff --git a/CMakeLists.txt b/CMakeLists.txt index b886c90..4a16b35 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -215,19 +215,19 @@ set(CMAKE_CPP_FLAGS "${CMAKE_CPP_FLAGS} ${DEBUG_SYMBOL}") if(CMAKE_SYSTEM_PROCESSOR MATCHES "(x86)|(X86)|(amd64)|(AMD64)") set(CMAKE_CPP_FLAGS "${CMAKE_CPP_FLAGS} -msse4 -msse4.2") endif() -set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${CMAKE_CPP_FLAGS} -O2 -pipe -Wall -W -fPIC -fstrict-aliasing -Wno-invalid-offsetof -Wno-unused-parameter -Wno-reserved-user-defined-literal -fno-omit-frame-pointer") -set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} ${CMAKE_CPP_FLAGS} -O2 -pipe -Wall -W -fPIC -fstrict-aliasing -Wno-unused-parameter -fno-omit-frame-pointer") +set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${CMAKE_CPP_FLAGS} -pipe -Wall -W -fPIC -fstrict-aliasing -Wno-invalid-offsetof -Wno-unused-parameter -Wno-reserved-user-defined-literal -fno-omit-frame-pointer") +set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} ${CMAKE_CPP_FLAGS} -pipe -Wall -W -fPIC -fstrict-aliasing -Wno-unused-parameter -fno-omit-frame-pointer") -macro(use_cxx11) +macro(use_cxx17) if(CMAKE_VERSION VERSION_LESS "3.1.3") - set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++11") + set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++17") else() - set(CMAKE_CXX_STANDARD 11) + set(CMAKE_CXX_STANDARD 17) set(CMAKE_CXX_STANDARD_REQUIRED ON) endif() -endmacro(use_cxx11) +endmacro(use_cxx17) -use_cxx11() +use_cxx17() add_subdirectory(src) if(WITH_TESTS) diff --git a/src/braft/ballot.cpp b/src/braft/ballot.cpp index b05c153..ec4951a 100644 --- a/src/braft/ballot.cpp +++ b/src/braft/ballot.cpp @@ -1,11 +1,11 @@ // Copyright (c) 2018 Baidu.com, Inc. All Rights Reserved -// +// // Licensed under the Apache License, Version 2.0 (the "License"); // you may not use this file except in compliance with the License. // You may obtain a copy of the License at -// +// // http://www.apache.org/licenses/LICENSE-2.0 -// +// // Unless required by applicable law or agreed to in writing, software // distributed under the License is distributed on an "AS IS" BASIS, // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. @@ -16,29 +16,29 @@ #include "braft/ballot.h" -namespace braft { +#include +#include "braft/configuration.h" -Ballot::Ballot() : _quorum(0), _old_quorum(0) {} -Ballot::~Ballot() {} +namespace braft { -void Ballot::init(const Configuration& conf, const Configuration* old_conf) { +void Ballot::init(const Configuration& conf, std::optional old_conf) { _peers.clear(); _old_peers.clear(); _quorum = 0; _old_quorum = 0; + CHECK_GT(conf.size(), 0); _peers.reserve(conf.size()); - for (Configuration::const_iterator - iter = conf.begin(); iter != conf.end(); ++iter) { + for (auto iter = conf.begin(); iter != conf.end(); ++iter) { _peers.push_back(*iter); } _quorum = _peers.size() / 2 + 1; - if (!old_conf) { + if (!old_conf.has_value()) { return; } _old_peers.reserve(old_conf->size()); - for (Configuration::const_iterator - iter = old_conf->begin(); iter != old_conf->end(); ++iter) { + for (Configuration::const_iterator iter = old_conf->begin(); + iter != old_conf->end(); ++iter) { _old_peers.push_back(*iter); } _old_quorum = _old_peers.size() / 2 + 1; @@ -54,11 +54,11 @@ Ballot::PosHint Ballot::grant(const PeerId& peer, PosHint hint) { } hint.pos0 = iter - _peers.begin(); } else { - hint.pos0 = -1; + hint.pos0 = 0; } if (_old_peers.empty()) { - hint.pos1 = -1; + hint.pos1 = 0; return hint; } @@ -71,14 +71,12 @@ Ballot::PosHint Ballot::grant(const PeerId& peer, PosHint hint) { } hint.pos1 = iter - _old_peers.begin(); } else { - hint.pos1 = -1; + hint.pos1 = 0; } return hint; } -void Ballot::grant(const PeerId& peer) { - grant(peer, PosHint()); -} +void Ballot::grant(const PeerId& peer) { grant(peer, PosHint()); } } // namespace braft diff --git a/src/braft/ballot.h b/src/braft/ballot.h index ae40c75..6e5a58e 100644 --- a/src/braft/ballot.h +++ b/src/braft/ballot.h @@ -1,11 +1,11 @@ // Copyright (c) 2018 Baidu.com, Inc. All Rights Reserved -// +// // Licensed under the Apache License, Version 2.0 (the "License"); // you may not use this file except in compliance with the License. // You may obtain a copy of the License at -// +// // http://www.apache.org/licenses/LICENSE-2.0 -// +// // Unless required by applicable law or agreed to in writing, software // distributed under the License is distributed on an "AS IS" BASIS, // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. @@ -14,63 +14,57 @@ // Authors: Zhangyi Chen(chenzhangyi01@baidu.com) -#ifndef BRAFT_BALLOT_H -#define BRAFT_BALLOT_H +#pragma once +#include +#include +#include #include "braft/configuration.h" namespace braft { class Ballot { -public: + public: struct PosHint { - PosHint() : pos0(-1), pos1(-1) {} - int pos0; - int pos1; + PosHint() : pos0(0), pos1(0) {} + size_t pos0; + size_t pos1; }; - Ballot(); - ~Ballot(); - void swap(Ballot& rhs) { - _peers.swap(rhs._peers); - std::swap(_quorum, rhs._quorum); - _old_peers.swap(rhs._old_peers); - std::swap(_old_quorum, rhs._old_quorum); - } - - void init(const Configuration& conf, const Configuration* old_conf); + Ballot() : _quorum(0), _old_quorum(0){}; + // FIXME(ehds): Remove optional. + // void init(const Configuration& conf, const Configuration& old_conf); + void init(const Configuration& conf, + std::optional old_conf); PosHint grant(const PeerId& peer, PosHint hint); void grant(const PeerId& peer); - bool granted() const { return _quorum <= 0 && _old_quorum <= 0; } -private: + bool granted() const { return quorum() <= 0 && old_quorum() <= 0; } + int quorum() const { return _quorum; } + int old_quorum() const { return _old_quorum; } + + private: struct UnfoundPeerId { UnfoundPeerId(const PeerId& peer_id) : peer_id(peer_id), found(false) {} PeerId peer_id; bool found; - bool operator==(const PeerId& id) const { - return peer_id == id; - } + bool operator==(const PeerId& id) const { return peer_id == id; } }; - std::vector::iterator find_peer( - const PeerId& peer, std::vector& peers, int pos_hint) { - if (pos_hint < 0 || pos_hint >= (int)peers.size() - || peers[pos_hint].peer_id != peer) { - for (std::vector::iterator - iter = peers.begin(); iter != peers.end(); ++iter) { - if (*iter == peer) { - return iter; - } - } - return peers.end(); + + using UnfoundPeerIds = std::vector; + using UnfoundPeerIter = UnfoundPeerIds::iterator; + + static UnfoundPeerIter find_peer(const PeerId& peer, UnfoundPeerIds& peers, + size_t pos_hint) { + if (pos_hint >= peers.size() || peers[pos_hint].peer_id != peer) { + return std::find(peers.begin(), peers.end(), peer); } return peers.begin() + pos_hint; } - std::vector _peers; + + UnfoundPeerIds _peers; int _quorum; - std::vector _old_peers; + UnfoundPeerIds _old_peers; int _old_quorum; }; -}; - -#endif //BRAFT_BALLOT_H +}; // namespace braft diff --git a/src/braft/ballot_box.cpp b/src/braft/ballot_box.cpp index 24151f4..c28f775 100644 --- a/src/braft/ballot_box.cpp +++ b/src/braft/ballot_box.cpp @@ -17,6 +17,8 @@ #include #include #include +#include +#include #include "braft/ballot_box.h" #include "braft/util.h" #include "braft/fsm_caller.h" @@ -121,12 +123,12 @@ int BallotBox::reset_pending_index(int64_t new_pending_index) { int BallotBox::append_pending_task(const Configuration& conf, const Configuration* old_conf, Closure* closure) { Ballot bl; - bl.init(conf, old_conf); + bl.init(conf, + old_conf == nullptr ? std::nullopt : std::make_optional(*old_conf)); BAIDU_SCOPED_LOCK(_mutex); CHECK(_pending_index > 0); - _pending_meta_queue.push_back(Ballot()); - _pending_meta_queue.back().swap(bl); + _pending_meta_queue.push_back(std::move(bl)); _closure_queue->append_pending_closure(closure); return 0; } diff --git a/src/braft/node.cpp b/src/braft/node.cpp index 8abad92..84b23c1 100644 --- a/src/braft/node.cpp +++ b/src/braft/node.cpp @@ -20,7 +20,9 @@ #include #include #include +#include +#include "braft/configuration.h" #include "braft/errno.pb.h" #include "braft/util.h" #include "braft/raft.h" @@ -2068,9 +2070,9 @@ void NodeImpl::apply(LogEntryAndClosure tasks[], size_t size) { entries.push_back(tasks[i].entry); entries.back()->id.term = _current_term; entries.back()->type = ENTRY_TYPE_DATA; - _ballot_box->append_pending_task(_conf.conf, - _conf.stable() ? NULL : &_conf.old_conf, - tasks[i].done); + _ballot_box->append_pending_task( + _conf.conf, _conf.stable() ? nullptr : &_conf.old_conf, + tasks[i].done); } _log_manager->append_entries(&entries, new LeaderStableClosure( @@ -3480,7 +3482,7 @@ void NodeImpl::get_leader_lease_status(LeaderLeaseStatus* lease_status) { void NodeImpl::VoteBallotCtx::init(NodeImpl* node, bool triggered) { reset(node); - _ballot.init(node->_conf.conf, node->_conf.stable() ? NULL : &(node->_conf.old_conf)); + _ballot.init(node->_conf.conf, std::optional() ); _triggered = triggered; } diff --git a/test/CMakeLists.txt b/test/CMakeLists.txt index c258168..244ff95 100644 --- a/test/CMakeLists.txt +++ b/test/CMakeLists.txt @@ -7,7 +7,7 @@ include_directories(${CMAKE_SOURCE_DIR}/test) set(CMAKE_CPP_FLAGS "-DGFLAGS_NS=${GFLAGS_NS}") set(CMAKE_CPP_FLAGS "${CMAKE_CPP_FLAGS} -D__const__=__unused__ -D_GNU_SOURCE -DUSE_SYMBOLIZE -DNO_TCMALLOC -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -D__STDC_CONSTANT_MACROS -DUNIT_TEST -g -Dprivate=public -Dprotected=public -D__STRICT_ANSI__ -include sstream_workaround.h") -set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${CMAKE_CPP_FLAGS} -O2 -pipe -Wall -W -fPIC -fstrict-aliasing -Wno-invalid-offsetof -Wno-unused-parameter -fno-omit-frame-pointer -Wno-unused-result") +set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${CMAKE_CPP_FLAGS} -pipe -Wall -W -fPIC -fstrict-aliasing -Wno-invalid-offsetof -Wno-unused-parameter -fno-omit-frame-pointer -Wno-unused-result") if (WITH_COVERAGE) if(CMAKE_C_COMPILER_ID STREQUAL "GNU") diff --git a/test/common.h b/test/common.h new file mode 100644 index 0000000..3cedf60 --- /dev/null +++ b/test/common.h @@ -0,0 +1,6 @@ +#pragma once +#ifdef private +#undef private +#endif +#include +#define private public diff --git a/test/test_ballot.cpp b/test/test_ballot.cpp index 5b10772..7a2e737 100644 --- a/test/test_ballot.cpp +++ b/test/test_ballot.cpp @@ -14,8 +14,10 @@ // Authors: Zhangyi Chen(chenzhangyi01@baidu.com) -#include +#include + #include "braft/ballot.h" +#include "common.h" class BallotTest : public testing::Test {}; @@ -29,17 +31,17 @@ TEST(BallotTest, sanity) { conf.add_peer(peer2); conf.add_peer(peer3); braft::Ballot bl; - bl.init(conf, NULL); - ASSERT_EQ(2, bl._quorum); - ASSERT_EQ(0, bl._old_quorum); + bl.init(conf, std::nullopt); + ASSERT_EQ(2, bl.quorum()); + ASSERT_EQ(0, bl.old_quorum()); bl.grant(peer1); - ASSERT_EQ(1, bl._quorum); + ASSERT_EQ(1, bl.quorum()); braft::Ballot::PosHint hint = bl.grant(peer1, braft::Ballot::PosHint()); - ASSERT_EQ(1, bl._quorum); + ASSERT_EQ(1, bl.quorum()); hint = bl.grant(peer1, hint); - ASSERT_EQ(1, bl._quorum); + ASSERT_EQ(1, bl.quorum()); hint = bl.grant(peer4, hint); - ASSERT_EQ(1, bl._quorum); + ASSERT_EQ(1, bl.quorum()); hint = bl.grant(peer2, hint); ASSERT_TRUE(bl.granted()); } @@ -54,27 +56,27 @@ TEST(BallotTest, joint_consensus_same_conf) { conf.add_peer(peer2); conf.add_peer(peer3); braft::Ballot bl; - bl.init(conf, &conf); - ASSERT_EQ(2, bl._quorum); - ASSERT_EQ(2, bl._old_quorum); + bl.init(conf, conf); + ASSERT_EQ(2, bl.quorum()); + ASSERT_EQ(2, bl.old_quorum()); bl.grant(peer1); - ASSERT_EQ(1, bl._quorum); - ASSERT_EQ(1, bl._old_quorum); + ASSERT_EQ(1, bl.quorum()); + ASSERT_EQ(1, bl.old_quorum()); braft::Ballot::PosHint hint = bl.grant(peer1, braft::Ballot::PosHint()); - ASSERT_EQ(1, bl._quorum); - ASSERT_EQ(1, bl._old_quorum); + ASSERT_EQ(1, bl.quorum()); + ASSERT_EQ(1, bl.old_quorum()); hint = bl.grant(peer1, hint); - ASSERT_EQ(1, bl._quorum); - ASSERT_EQ(1, bl._old_quorum); + ASSERT_EQ(1, bl.quorum()); + ASSERT_EQ(1, bl.old_quorum()); hint = bl.grant(peer4, hint); - ASSERT_EQ(1, bl._quorum); - ASSERT_EQ(1, bl._old_quorum); + ASSERT_EQ(1, bl.quorum()); + ASSERT_EQ(1, bl.old_quorum()); ASSERT_FALSE(bl.granted()); hint = bl.grant(peer2, hint); ASSERT_TRUE(bl.granted()); hint = bl.grant(peer3, hint); - ASSERT_EQ(-1, bl._quorum); - ASSERT_EQ(-1, bl._old_quorum); + ASSERT_EQ(-1, bl.quorum()); + ASSERT_EQ(-1, bl.old_quorum()); } TEST(BallotTest, joint_consensus_different_conf) { @@ -92,12 +94,12 @@ TEST(BallotTest, joint_consensus_different_conf) { conf2.add_peer(peer3); conf2.add_peer(peer4); braft::Ballot bl; - bl.init(conf, &conf2); + bl.init(conf, conf2); bl.grant(peer1); bl.grant(peer2); ASSERT_FALSE(bl.granted()); - ASSERT_EQ(0, bl._quorum); - ASSERT_EQ(1, bl._old_quorum); + ASSERT_EQ(0, bl.quorum()); + ASSERT_EQ(1, bl.old_quorum()); bl.grant(peer4); ASSERT_TRUE(bl.granted()); } diff --git a/test/test_ballot_box.cpp b/test/test_ballot_box.cpp index 4b907cf..efc0f02 100644 --- a/test/test_ballot_box.cpp +++ b/test/test_ballot_box.cpp @@ -3,12 +3,14 @@ // Author: Zhangyi Chen (chenzhangyi01@baidu.com) // Date: 2016/02/03 15:59:18 -#include -#include #include + +#include + #include "braft/ballot_box.h" #include "braft/configuration.h" #include "braft/fsm_caller.h" +#include "common.h" class BallotBoxTest : public testing::Test { protected: diff --git a/test/test_checksum.cpp b/test/test_checksum.cpp index 110274c..5666c0b 100644 --- a/test/test_checksum.cpp +++ b/test/test_checksum.cpp @@ -3,10 +3,11 @@ // Author: Zhangyi Chen (chenzhangyi01@baidu.com) // Date: 2016/04/11 12:15:37 -#include #include #include +#include "common.h" + class ChecksumTest : public testing::Test { protected: void SetUp() {} diff --git a/test/test_cli.cpp b/test/test_cli.cpp index ed816ab..5cd250a 100644 --- a/test/test_cli.cpp +++ b/test/test_cli.cpp @@ -3,13 +3,14 @@ // Author: Zhangyi Chen (chenzhangyi01@baidu.com) // Date: 2018/01/12 12:56:17 -#include -#include -#include #include -#include "braft/raft.h" +#include +#include + #include "braft/cli.h" #include "braft/node.h" +#include "braft/raft.h" +#include "common.h" class CliTest : public testing::Test { public: diff --git a/test/test_configuration.cpp b/test/test_configuration.cpp index 9461128..b933367 100644 --- a/test/test_configuration.cpp +++ b/test/test_configuration.cpp @@ -16,10 +16,11 @@ * ===================================================================================== */ -#include #include -#include "braft/raft.h" + #include "braft/configuration_manager.h" +#include "braft/raft.h" +#include "common.h" class TestUsageSuits : public testing::Test { protected: diff --git a/test/test_file_service.cpp b/test/test_file_service.cpp index 0f805b2..7703c76 100644 --- a/test/test_file_service.cpp +++ b/test/test_file_service.cpp @@ -3,16 +3,16 @@ // Author: Zhangyi Chen (chenzhangyi01@baidu.com) // Date: 2015/11/06 15:40:45 -#include -#include -#include +#include #include +#include +#include -#include #include "braft/file_service.h" -#include "braft/util.h" -#include "braft/remote_file_copier.h" #include "braft/file_system_adaptor.h" +#include "braft/remote_file_copier.h" +#include "braft/util.h" +#include "common.h" namespace braft { DECLARE_bool(raft_file_check_hole); diff --git a/test/test_file_system_adaptor.cpp b/test/test_file_system_adaptor.cpp index f5da03c..9670504 100644 --- a/test/test_file_system_adaptor.cpp +++ b/test/test_file_system_adaptor.cpp @@ -4,8 +4,8 @@ // Author: ZhengPengFei (zhengpengfei@baidu.com) // Date: 2017/06/16 10:29:05 -#include #include "braft/file_system_adaptor.h" +#include "common.h" class TestFileSystemAdaptorSuits : public testing::Test { protected: diff --git a/test/test_fsm_caller.cpp b/test/test_fsm_caller.cpp index bb3c186..cd2a0ff 100644 --- a/test/test_fsm_caller.cpp +++ b/test/test_fsm_caller.cpp @@ -3,14 +3,15 @@ // Author: Zhangyi Chen (chenzhangyi01@baidu.com) // Date: 2015/12/01 17:03:46 -#include -#include #include +#include + +#include "braft/configuration.h" #include "braft/fsm_caller.h" -#include "braft/raft.h" #include "braft/log.h" -#include "braft/configuration.h" #include "braft/log_manager.h" +#include "braft/raft.h" +#include "common.h" class FSMCallerTest : public testing::Test { protected: diff --git a/test/test_fsync.cpp b/test/test_fsync.cpp index 96bbdbf..604ef9a 100644 --- a/test/test_fsync.cpp +++ b/test/test_fsync.cpp @@ -3,14 +3,15 @@ // Author: Zhangyi Chen (chenzhangyi01@baidu.com) // Date: 2016/02/23 16:22:15 -#include -#include -#include -#include -#include #include -#include #include +#include +#include +#include +#include +#include + +#include "common.h" class FsyncTest : public testing::Test { }; diff --git a/test/test_leader_lease.cpp b/test/test_leader_lease.cpp index d296b8b..005c9d1 100644 --- a/test/test_leader_lease.cpp +++ b/test/test_leader_lease.cpp @@ -14,12 +14,13 @@ // Authors: Pengfei Zheng (zhengpengfei@baidu.com) -#include #include -#include "braft/util.h" -#include "braft/node.h" -#include "braft/lease.h" + #include "../test/util.h" +#include "braft/lease.h" +#include "braft/node.h" +#include "braft/util.h" +#include "common.h" namespace braft { DECLARE_bool(raft_enable_leader_lease); diff --git a/test/test_log.cpp b/test/test_log.cpp index 25d046f..537b492 100644 --- a/test/test_log.cpp +++ b/test/test_log.cpp @@ -4,20 +4,21 @@ // Author: WangYao (fisherman), wangyao02@baidu.com // Date: 2015/10/08 17:00:05 -#include -#include -#include -#include #include #include -#include -#include #include -#include +#include +#include #include -#include "braft/util.h" +#include +#include +#include +#include + #include "braft/log.h" #include "braft/storage.h" +#include "braft/util.h" +#include "common.h" namespace braft { DECLARE_bool(raft_trace_append_entry_latency); diff --git a/test/test_log_entry.cpp b/test/test_log_entry.cpp index 600e274..bae8a5d 100644 --- a/test/test_log_entry.cpp +++ b/test/test_log_entry.cpp @@ -16,10 +16,11 @@ * ===================================================================================== */ -#include -#include #include +#include + #include "braft/log_entry.h" +#include "common.h" class TestUsageSuits : public testing::Test { protected: diff --git a/test/test_log_manager.cpp b/test/test_log_manager.cpp index e555d4a..bc8883c 100644 --- a/test/test_log_manager.cpp +++ b/test/test_log_manager.cpp @@ -3,16 +3,15 @@ // Author: Zhangyi Chen (chenzhangyi01@baidu.com) // Date: 2015/11/24 16:30:49 -#include - +#include +#include #include #include -#include -#include -#include "braft/log_manager.h" #include "braft/configuration.h" #include "braft/log.h" +#include "braft/log_manager.h" +#include "common.h" class LogManagerTest : public testing::Test { protected: diff --git a/test/test_memory_storage.cpp b/test/test_memory_storage.cpp index 88e16ce..779abdb 100644 --- a/test/test_memory_storage.cpp +++ b/test/test_memory_storage.cpp @@ -3,8 +3,8 @@ // Author: qinduohao@baidu.com // Date: 2017/05/23 -#include #include "braft/memory_log.h" +#include "common.h" namespace braft { extern void global_init_once_or_die(); diff --git a/test/test_meta.cpp b/test/test_meta.cpp index 903a645..268b538 100644 --- a/test/test_meta.cpp +++ b/test/test_meta.cpp @@ -1,8 +1,9 @@ -#include -#include #include +#include + #include "braft/raft.h" #include "braft/raft_meta.h" +#include "common.h" namespace braft { extern void global_init_once_or_die(); diff --git a/test/test_node.cpp b/test/test_node.cpp index 10cc01a..11668d4 100644 --- a/test/test_node.cpp +++ b/test/test_node.cpp @@ -4,22 +4,24 @@ // Author: WangYao (fisherman), wangyao02@baidu.com // Date: 2015/10/08 17:00:05 -#include -#include -#include -#include -#include -#include #include -#include -#include -#include -#include #include #include #include +#include +#include +#include +#include +#include +#include +#include +#include + +#include #include + #include "../test/util.h" +#include "common.h" namespace braft { extern bvar::Adder g_num_nodes; diff --git a/test/test_protobuf_file.cpp b/test/test_protobuf_file.cpp index fdb6830..a130f3a 100644 --- a/test/test_protobuf_file.cpp +++ b/test/test_protobuf_file.cpp @@ -3,7 +3,7 @@ * * Filename: test_protobuf_file.cpp * - * Description: + * Description: * * Version: 1.0 * Created: 2015/09/22 19:48:31 @@ -15,8 +15,8 @@ * * ===================================================================================== */ +#include "common.h" -#include #include "braft/local_storage.pb.h" #include "braft/protobuf_file.h" diff --git a/test/test_repeated_timer_task.cpp b/test/test_repeated_timer_task.cpp index 2e15710..df29c9d 100644 --- a/test/test_repeated_timer_task.cpp +++ b/test/test_repeated_timer_task.cpp @@ -4,7 +4,8 @@ // Author: Zhangyi Chen (chenzhangyi01@baidu.com) // Date: 2016/11/02 21:12:26 -#include +#include "common.h" + #include "braft/repeated_timer_task.h" class RepeatedTimerTaskTest : public testing::Test { diff --git a/test/test_route_table.cpp b/test/test_route_table.cpp index a05b26b..e8eb788 100644 --- a/test/test_route_table.cpp +++ b/test/test_route_table.cpp @@ -1,9 +1,11 @@ -#include "braft/configuration.h" -#include "braft/route_table.h" #include -#include + #include + #include "../test/util.h" +#include "braft/configuration.h" +#include "braft/route_table.h" +#include "common.h" class RouteTableTest : public testing::Test { void SetUp() { diff --git a/test/test_snapshot.cpp b/test/test_snapshot.cpp index 3b3a98c..23648a0 100644 --- a/test/test_snapshot.cpp +++ b/test/test_snapshot.cpp @@ -4,17 +4,18 @@ // Author: WangYao (fisherman), wangyao02@baidu.com // Date: 2015/10/08 17:00:05 -#include -#include -#include +#include #include +#include #include -#include -#include "braft/snapshot.h" -#include "braft/raft.h" -#include "braft/util.h" +#include + #include "braft/local_file_meta.pb.h" +#include "braft/raft.h" +#include "braft/snapshot.h" #include "braft/snapshot_throttle.h" +#include "braft/util.h" +#include "common.h" #include "memory_file_system_adaptor.h" namespace logging { diff --git a/test/test_snapshot_executor.cpp b/test/test_snapshot_executor.cpp index 7d63bb0..b564215 100644 --- a/test/test_snapshot_executor.cpp +++ b/test/test_snapshot_executor.cpp @@ -3,13 +3,14 @@ // Author: Zhangyi Chen (chenzhangyi01@baidu.com) // Date: 2016/07/27 20:13:42 -#include -#include #include -#include "braft/snapshot_executor.h" +#include + #include "braft/fsm_caller.h" -#include "braft/util.h" #include "braft/raft.h" +#include "braft/snapshot_executor.h" +#include "braft/util.h" +#include "common.h" namespace braft { diff --git a/test/test_storage.cpp b/test/test_storage.cpp index c9f7aea..7ea0967 100644 --- a/test/test_storage.cpp +++ b/test/test_storage.cpp @@ -3,7 +3,8 @@ // Author: Zhangyi Chen (chenzhangyi01@baidu.com) // Date: 2016/03/29 10:35:06 -#include +#include + #include "braft/storage.h" namespace braft { diff --git a/test/test_throttle.cpp b/test/test_throttle.cpp index 94b35a9..7248b36 100644 --- a/test/test_throttle.cpp +++ b/test/test_throttle.cpp @@ -4,11 +4,12 @@ // Author: Xiong Kai (xiongkai@baidu.com) // Date: 2017/09/07 14:06:13 -#include #include + #include "braft/raft.h" -#include "braft/util.h" #include "braft/snapshot_throttle.h" +#include "braft/util.h" +#include "common.h" class TestUsageSuits : public testing::Test { protected: diff --git a/test/test_util.cpp b/test/test_util.cpp index 731d9fa..6ac9a78 100644 --- a/test/test_util.cpp +++ b/test/test_util.cpp @@ -4,14 +4,14 @@ // Author: WangYao (fisherman), wangyao02@baidu.com // Date: 2015/10/08 17:00:05 -#include +#include #include -#include -#include #include -#include +#include +#include #include "braft/util.h" +#include "common.h" class TestUsageSuits : public testing::Test { protected: diff --git a/test/util.h b/test/util.h index 4730fbd..836065d 100644 --- a/test/util.h +++ b/test/util.h @@ -18,11 +18,13 @@ #define PUBLIC_RAFT_TEST_UTIL_H #include -#include "braft/node.h" + #include "braft/enum.pb.h" #include "braft/errno.pb.h" -#include "braft/snapshot_throttle.h" +#include "braft/node.h" #include "braft/snapshot_executor.h" +#include "braft/snapshot_throttle.h" +#include "common.h" using namespace braft; bool g_dont_print_apply_log = false;