This is an automated email from the ASF dual-hosted git repository. alexey pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/kudu.git
commit 5e37e599e7d13be1d88bd70de5aab00b6f8a93e9 Author: Andrew Wong <[email protected]> AuthorDate: Mon Oct 28 19:20:32 2019 -0700 test: fix tablet_server-test on macOS Mac doesn't support the log block manager; this updates the test parameterization to only run with the appropriate block manager. Change-Id: I30c6c9cc53bfba817f79ed980e74283283c62633 Reviewed-on: http://gerrit.cloudera.org:8080/14563 Reviewed-by: Alexey Serbin <[email protected]> Tested-by: Andrew Wong <[email protected]> --- src/kudu/tserver/tablet_server-test.cc | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/kudu/tserver/tablet_server-test.cc b/src/kudu/tserver/tablet_server-test.cc index 69dfff4..cd5dcf8 100644 --- a/src/kudu/tserver/tablet_server-test.cc +++ b/src/kudu/tserver/tablet_server-test.cc @@ -57,6 +57,7 @@ #include "kudu/consensus/metadata.pb.h" #include "kudu/consensus/raft_consensus.h" #include "kudu/fs/block_id.h" +#include "kudu/fs/block_manager.h" #include "kudu/fs/data_dirs.h" #include "kudu/fs/fs-test-util.h" #include "kudu/fs/fs.pb.h" @@ -121,6 +122,7 @@ using google::protobuf::util::MessageDifferencer; using kudu::clock::Clock; using kudu::clock::HybridClock; using kudu::consensus::ConsensusStatePB; +using kudu::fs::BlockManager; using kudu::fs::CreateCorruptBlock; using kudu::fs::DataDirManager; using kudu::pb_util::SecureDebugString; @@ -706,7 +708,7 @@ TEST_P(TabletServerDiskSpaceTest, TestFullGroupAddsDir) { } INSTANTIATE_TEST_CASE_P(BlockManager, TabletServerDiskSpaceTest, - ::testing::Values("log", "file")); + ::testing::ValuesIn(BlockManager::block_manager_types())); enum class ErrorType { DISK_FAILURE,
