From 48b53441fd740b46c2d96c648781e80e74b8b9fe Mon Sep 17 00:00:00 2001 From: Andrew Kryczka Date: Mon, 20 Dec 2021 10:27:35 -0800 Subject: [PATCH] Skip MemoryAllocatorTest in LITE mode (#9318) Summary: The tests rely on `CreateFromString()`, which returns `Status::NotSupported()` when these tests attempt to create non-default allocators. Pull Request resolved: https://github.com/facebook/rocksdb/pull/9318 Reviewed By: riversand963 Differential Revision: D33238405 Pulled By: ajkr fbshipit-source-id: d2974e2341f1494f5f7cd07b73f2dbd0d502fc7c --- memory/memory_allocator_test.cc | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/memory/memory_allocator_test.cc b/memory/memory_allocator_test.cc index 78dc221b8f..3cc14840ec 100644 --- a/memory/memory_allocator_test.cc +++ b/memory/memory_allocator_test.cc @@ -17,6 +17,11 @@ #include "utilities/memory_allocators.h" namespace ROCKSDB_NAMESPACE { + +// TODO: the tests do not work in LITE mode due to relying on +// `CreateFromString()` to create non-default memory allocators. +#ifndef ROCKSDB_LITE + class MemoryAllocatorTest : public testing::Test, public ::testing::WithParamInterface> { @@ -228,6 +233,8 @@ INSTANTIATE_TEST_CASE_P( JemallocNodumpAllocator::IsSupported()))); #endif // ROCKSDB_JEMALLOC +#endif // ROCKSDB_LITE + } // namespace ROCKSDB_NAMESPACE int main(int argc, char** argv) {