Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add grpc stop for flip service in homestore. #280

Merged
merged 1 commit into from
Jan 19, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion conanfile.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@

class HomestoreConan(ConanFile):
name = "homestore"
version = "5.0.2"
version = "5.0.3"
homepage = "https://github.com/eBay/Homestore"
description = "HomeStore Storage Engine"
topics = ("ebay", "nublox")
Expand Down
2 changes: 1 addition & 1 deletion src/lib/checkpoint/cp_mgr.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -275,7 +275,7 @@ void CPManager::start_cp_thread() {
auto ctx = std::make_shared< Context >();

// Start a reactor with 9 fibers (8 for sync io)
iomanager.create_reactor("cp_io", iomgr::INTERRUPT_LOOP, 8u, [this, &ctx](bool is_started) {
iomanager.create_reactor("cp_io", iomgr::INTERRUPT_LOOP, 8u, [this, ctx](bool is_started) {
if (is_started) {
{
std::unique_lock< std::mutex > lk{ctx->mtx};
Expand Down
6 changes: 5 additions & 1 deletion src/lib/homestore.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -240,7 +240,7 @@ void HomeStore::shutdown() {

if (has_index_service()) {
m_index_service->stop();
// m_index_service.reset();
// m_index_service.reset();
}

if (has_log_service()) {
Expand All @@ -258,6 +258,10 @@ void HomeStore::shutdown() {
m_dev_mgr->close_devices();
m_dev_mgr.reset();

#ifdef _PRERELEASE
flip::Flip::instance().stop_rpc_server();
#endif

HomeStore::reset_instance();
LOGINFO("Homestore is completed its shutdown");
}
Expand Down
9 changes: 5 additions & 4 deletions src/tests/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -36,8 +36,8 @@ if (${build_nonio_tests})
set(TEST_MEMBTREE_SOURCE_FILES test_mem_btree.cpp)
add_executable(test_mem_btree ${TEST_MEMBTREE_SOURCE_FILES})
target_link_libraries(test_mem_btree ${COMMON_TEST_DEPS} GTest::gtest)
# add_test(NAME MemBtree COMMAND test_mem_btree)
# set_tests_properties(MemBtree PROPERTIES TIMEOUT 180)
add_test(NAME MemBtree COMMAND test_mem_btree)
set_tests_properties(MemBtree PROPERTIES TIMEOUT 600)

add_executable(test_blk_read_tracker)
target_sources(test_blk_read_tracker PRIVATE test_blk_read_tracker.cpp ../lib/blkdata_svc/blk_read_tracker.cpp ../lib/blkalloc/blk.cpp)
Expand Down Expand Up @@ -72,8 +72,9 @@ if (${io_tests})
set(TEST_INDEXBTREE_SOURCE_FILES test_index_btree.cpp)
add_executable(test_index_btree ${TEST_INDEXBTREE_SOURCE_FILES})
target_link_libraries(test_index_btree homestore ${COMMON_TEST_DEPS} GTest::gtest)
# add_test(NAME IndexBtree COMMAND test_index_btree)
# set_property(TEST IndexBtree PROPERTY ENVIRONMENT "ASAN_OPTIONS=detect_stack_use_after_return=true")
add_test(NAME IndexBtree COMMAND test_index_btree)
set_property(TEST IndexBtree PROPERTY ENVIRONMENT "ASAN_OPTIONS=detect_stack_use_after_return=true")
set_tests_properties(IndexBtree PROPERTIES TIMEOUT 600)

add_executable(test_data_service)
target_sources(test_data_service PRIVATE test_data_service.cpp)
Expand Down
4 changes: 1 addition & 3 deletions src/tests/test_index_btree.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,6 @@ SISL_LOGGING_DECL(test_index_btree)

std::vector< std::string > test_common::HSTestHelper::s_dev_names;


// TODO Add tests to do write,remove after recovery.
// TODO Test with var len key with io mgr page size is 512.

Expand Down Expand Up @@ -121,7 +120,7 @@ struct BtreeTest : public BtreeTestHelper< TestType >, public ::testing::Test {
}
};

using BtreeTypes = testing::Types< FixedLenBtree, VarKeySizeBtree, VarValueSizeBtree, VarObjSizeBtree >;
using BtreeTypes = testing::Types< FixedLenBtree /* , VarKeySizeBtree, VarValueSizeBtree, VarObjSizeBtree */ >;
sanebay marked this conversation as resolved.
Show resolved Hide resolved

TYPED_TEST_SUITE(BtreeTest, BtreeTypes);

Expand Down Expand Up @@ -431,7 +430,6 @@ struct BtreeConcurrentTest : public BtreeTestHelper< TestType >, public ::testin
BtreeConcurrentTest* m_test;
};


BtreeConcurrentTest() : testing::Test() { this->m_is_multi_threaded = true; }

void SetUp() override {
Expand Down