diff --git a/test/performance/msys_handle_leakage/unit-cppapi-consolidation-with-timestamps.cc b/test/performance/msys_handle_leakage/unit-cppapi-consolidation-with-timestamps.cc index 51f9c6cc846..a7dd7143b05 100644 --- a/test/performance/msys_handle_leakage/unit-cppapi-consolidation-with-timestamps.cc +++ b/test/performance/msys_handle_leakage/unit-cppapi-consolidation-with-timestamps.cc @@ -5,7 +5,7 @@ * * The MIT License * - * @copyright Copyright (c) 2022 TileDB Inc. + * @copyright Copyright (c) 2022-2026 TileDB Inc. * * Permission is hereby granted, free of charge, to any person obtaining a copy * of this software and associated documentation files (the "Software"), to deal @@ -104,12 +104,8 @@ struct ConsolidationWithTimestampsFx { }; ConsolidationWithTimestampsFx::ConsolidationWithTimestampsFx() - : vfs_(ctx_) { - Config config; - config.set("sm.consolidation.buffer_size", "1000"); - ctx_ = Context(config); - sm_ = ctx_.ptr().get()->storage_manager(); - vfs_ = VFS(ctx_); + : vfs_(ctx_) + , sm_(ctx_.ptr().get()->storage_manager()) { } ConsolidationWithTimestampsFx::~ConsolidationWithTimestampsFx() { @@ -117,7 +113,6 @@ ConsolidationWithTimestampsFx::~ConsolidationWithTimestampsFx() { void ConsolidationWithTimestampsFx::set_legacy() { Config config; - config.set("sm.consolidation.buffer_size", "1000"); config.set("sm.query.sparse_global_order.reader", "legacy"); config.set("sm.query.sparse_unordered_with_dups.reader", "legacy"); diff --git a/test/src/test-capi-consolidation-plan.cc b/test/src/test-capi-consolidation-plan.cc index f190a41e1e8..fa745dc9d9b 100644 --- a/test/src/test-capi-consolidation-plan.cc +++ b/test/src/test-capi-consolidation-plan.cc @@ -5,7 +5,7 @@ * * The MIT License * - * @copyright Copyright (c) 2023 TileDB, Inc. + * @copyright Copyright (c) 2023-2026 TileDB, Inc. * * Permission is hereby granted, free of charge, to any person obtaining a copy * of this software and associated documentation files (the "Software"), to deal @@ -68,9 +68,6 @@ struct ConsolidationPlanFx { }; ConsolidationPlanFx::ConsolidationPlanFx() { - Config config; - config.set("sm.consolidation.buffer_size", "1000"); - vfs_test_setup_.update_config(config.ptr().get()); ctx_c_ = vfs_test_setup_.ctx_c; ctx_ = vfs_test_setup_.ctx(); array_name_ = vfs_test_setup_.array_uri("test_consolidation_plan_array"); diff --git a/test/src/test-cppapi-consolidation-plan.cc b/test/src/test-cppapi-consolidation-plan.cc index 9b9e3f8ab1e..42ffc6590bb 100644 --- a/test/src/test-cppapi-consolidation-plan.cc +++ b/test/src/test-cppapi-consolidation-plan.cc @@ -5,7 +5,7 @@ * * The MIT License * - * @copyright Copyright (c) 2023 TileDB, Inc. + * @copyright Copyright (c) 2023-2026 TileDB, Inc. * * Permission is hereby granted, free of charge, to any person obtaining a copy * of this software and associated documentation files (the "Software"), to deal @@ -84,7 +84,6 @@ struct CppConsolidationPlanFx { CppConsolidationPlanFx::CppConsolidationPlanFx() : vfs_(ctx_) { - cfg_.set("sm.consolidation.buffer_size", "1000"); ctx_ = Context(cfg_); vfs_ = VFS(ctx_); diff --git a/test/src/unit-capi-config.cc b/test/src/unit-capi-config.cc index f286faa91f5..59355a454c4 100644 --- a/test/src/unit-capi-config.cc +++ b/test/src/unit-capi-config.cc @@ -5,7 +5,7 @@ * * The MIT License * - * @copyright Copyright (c) 2017-2025 TileDB Inc. + * @copyright Copyright (c) 2017-2026 TileDB Inc. * * Permission is hereby granted, free of charge, to any person obtaining a copy * of this software and associated documentation files (the "Software"), to deal @@ -254,7 +254,6 @@ void check_save_to_file() { ss << "sm.compute_concurrency_level " << std::thread::hardware_concurrency() << "\n"; ss << "sm.consolidation.amplification 1.0\n"; - ss << "sm.consolidation.buffer_size 50000000\n"; ss << "sm.consolidation.max_fragment_size " << std::to_string(UINT64_MAX) << "\n"; ss << "sm.consolidation.mode fragments\n"; @@ -686,7 +685,6 @@ TEST_CASE("C API: Test config iter", "[capi][config]") { all_param_values["sm.consolidation.purge_deleted_cells"] = "false"; all_param_values["sm.consolidation.step_min_frags"] = "4294967295"; all_param_values["sm.consolidation.step_max_frags"] = "4294967295"; - all_param_values["sm.consolidation.buffer_size"] = "50000000"; all_param_values["sm.consolidation.max_fragment_size"] = std::to_string(UINT64_MAX); all_param_values["sm.consolidation.step_size_ratio"] = "0.0"; diff --git a/test/src/unit-capi-consolidation.cc b/test/src/unit-capi-consolidation.cc index e1907946285..a7692425159 100644 --- a/test/src/unit-capi-consolidation.cc +++ b/test/src/unit-capi-consolidation.cc @@ -5,7 +5,7 @@ * * The MIT License * - * @copyright Copyright (c) 2017-2021 TileDB Inc. + * @copyright Copyright (c) 2017-2026 TileDB Inc. * * Permission is hereby granted, free of charge, to any person obtaining a copy * of this software and associated documentation files (the "Software"), to deal @@ -123,8 +123,7 @@ struct ConsolidationFx { uint64_t start = 0, uint64_t end = UINT64_MAX); void consolidate_sparse_heterogeneous(); - void consolidate_sparse_string( - uint64_t buffer_size = 10000, bool error_expected = false); + void consolidate_sparse_string(bool error_expected = false); void vacuum_dense( const std::string& mode = "fragments", bool expect_fail = false); void vacuum_sparse( @@ -4464,10 +4463,6 @@ void ConsolidationFx::consolidate_dense( REQUIRE(rc == TILEDB_OK); REQUIRE(err == nullptr); - rc = tiledb_config_set(cfg, "sm.consolidation.buffer_size", "10000", &err); - REQUIRE(rc == TILEDB_OK); - REQUIRE(err == nullptr); - if (encryption_type_ != TILEDB_NO_ENCRYPTION) { std::string encryption_type_string = encryption_type_str((tiledb::sm::EncryptionType)encryption_type_); @@ -4505,10 +4500,6 @@ void ConsolidationFx::consolidate_sparse( REQUIRE(rc == TILEDB_OK); REQUIRE(err == nullptr); - rc = tiledb_config_set(cfg, "sm.consolidation.buffer_size", "10000", &err); - REQUIRE(rc == TILEDB_OK); - REQUIRE(err == nullptr); - if (encryption_type_ != TILEDB_NO_ENCRYPTION) { std::string encryption_type_string = encryption_type_str((tiledb::sm::EncryptionType)encryption_type_); @@ -4527,17 +4518,11 @@ void ConsolidationFx::consolidate_sparse( void ConsolidationFx::consolidate_sparse_heterogeneous() { int rc; tiledb_config_t* cfg; - tiledb_error_t* err = nullptr; - - rc = tiledb_config_alloc(&cfg, &err); - REQUIRE(rc == TILEDB_OK); - REQUIRE(err == nullptr); - - rc = tiledb_config_set(cfg, "sm.consolidation.buffer_size", "10000", &err); - REQUIRE(rc == TILEDB_OK); - REQUIRE(err == nullptr); - if (encryption_type_ != TILEDB_NO_ENCRYPTION) { + tiledb_error_t* err = nullptr; + rc = tiledb_config_alloc(&cfg, &err); + REQUIRE(rc == TILEDB_OK); + REQUIRE(err == nullptr); std::string encryption_type_string = encryption_type_str((tiledb::sm::EncryptionType)encryption_type_); rc = tiledb_config_set( @@ -4550,32 +4535,23 @@ void ConsolidationFx::consolidate_sparse_heterogeneous() { ctx_, sparse_heterogeneous_array_uri_.c_str(), cfg); tiledb_config_free(&cfg); } else { - rc = tiledb_array_consolidate( - ctx_, sparse_heterogeneous_array_uri_.c_str(), cfg); + cfg = nullptr; } - tiledb_config_free(&cfg); + rc = tiledb_array_consolidate( + ctx_, sparse_heterogeneous_array_uri_.c_str(), cfg); REQUIRE(rc == TILEDB_OK); + tiledb_config_free(&cfg); } -void ConsolidationFx::consolidate_sparse_string( - uint64_t buffer_size, bool error_expected) { +void ConsolidationFx::consolidate_sparse_string(bool error_expected) { int rc; - tiledb_config_t* cfg; - tiledb_error_t* err = nullptr; - - rc = tiledb_config_alloc(&cfg, &err); - REQUIRE(rc == TILEDB_OK); - REQUIRE(err == nullptr); - - auto buffer_size_string = std::to_string(buffer_size); - rc = tiledb_config_set( - cfg, "sm.consolidation.buffer_size", buffer_size_string.c_str(), &err); - REQUIRE(rc == TILEDB_OK); - REQUIRE(err == nullptr); - if (encryption_type_ != TILEDB_NO_ENCRYPTION) { + tiledb_error_t* err = nullptr; + rc = tiledb_config_alloc(&cfg, &err); + REQUIRE(rc == TILEDB_OK); + REQUIRE(err == nullptr); std::string encryption_type_string = encryption_type_str((tiledb::sm::EncryptionType)encryption_type_); rc = tiledb_config_set( @@ -4587,11 +4563,11 @@ void ConsolidationFx::consolidate_sparse_string( rc = tiledb_array_consolidate(ctx_, sparse_string_array_uri_.c_str(), cfg); tiledb_config_free(&cfg); } else { - rc = tiledb_array_consolidate(ctx_, sparse_string_array_uri_.c_str(), cfg); + cfg = nullptr; } - - tiledb_config_free(&cfg); + rc = tiledb_array_consolidate(ctx_, sparse_string_array_uri_.c_str(), cfg); REQUIRE(rc == (error_expected ? TILEDB_ERR : TILEDB_OK)); + tiledb_config_free(&cfg); } void ConsolidationFx::vacuum_dense(const std::string& mode, bool expect_fail) { @@ -4956,20 +4932,6 @@ TEST_CASE_METHOD( REQUIRE(rc == TILEDB_OK); REQUIRE(error == nullptr); - // Test buffer size - rc = tiledb_config_set(config, "sm.consolidation.buffer_size", "-1", &error); - REQUIRE(rc == TILEDB_ERR); - REQUIRE(error != nullptr); - tiledb_error_free(&error); - rc = tiledb_config_set(config, "sm.consolidation.buffer_size", "1.5", &error); - REQUIRE(rc == TILEDB_ERR); - REQUIRE(error != nullptr); - tiledb_error_free(&error); - rc = tiledb_config_set( - config, "sm.consolidation.buffer_size", "10000000", &error); - REQUIRE(rc == TILEDB_OK); - REQUIRE(error == nullptr); - // Test purge deleted cells rc = tiledb_config_set( config, "sm.consolidation.purge_deleted_cells", "1", &error); @@ -5099,11 +5061,6 @@ TEST_CASE_METHOD( REQUIRE(rc == TILEDB_OK); REQUIRE(error == nullptr); - rc = tiledb_config_set( - config, "sm.consolidation.buffer_size", "10000", &error); - REQUIRE(rc == TILEDB_OK); - REQUIRE(error == nullptr); - // Consolidate rc = tiledb_array_consolidate(ctx_, dense_vector_uri_.c_str(), config); CHECK(rc == TILEDB_OK); @@ -5162,11 +5119,6 @@ TEST_CASE_METHOD( REQUIRE(rc == TILEDB_OK); REQUIRE(error == nullptr); - rc = tiledb_config_set( - config, "sm.consolidation.buffer_size", "10000", &error); - REQUIRE(rc == TILEDB_OK); - REQUIRE(error == nullptr); - // Consolidate rc = tiledb_array_consolidate(ctx_, dense_vector_uri_.c_str(), config); CHECK(rc == TILEDB_OK); @@ -5225,11 +5177,6 @@ TEST_CASE_METHOD( REQUIRE(rc == TILEDB_OK); REQUIRE(error == nullptr); - rc = tiledb_config_set( - config, "sm.consolidation.buffer_size", "10000", &error); - REQUIRE(rc == TILEDB_OK); - REQUIRE(error == nullptr); - // Consolidate rc = tiledb_array_consolidate(ctx_, dense_vector_uri_.c_str(), config); CHECK(rc == TILEDB_OK); @@ -5288,11 +5235,6 @@ TEST_CASE_METHOD( REQUIRE(rc == TILEDB_OK); REQUIRE(error == nullptr); - rc = tiledb_config_set( - config, "sm.consolidation.buffer_size", "10000", &error); - REQUIRE(rc == TILEDB_OK); - REQUIRE(error == nullptr); - // Consolidate rc = tiledb_array_consolidate(ctx_, dense_vector_uri_.c_str(), config); CHECK(rc == TILEDB_OK); @@ -5351,11 +5293,6 @@ TEST_CASE_METHOD( REQUIRE(rc == TILEDB_OK); REQUIRE(error == nullptr); - rc = tiledb_config_set( - config, "sm.consolidation.buffer_size", "10000", &error); - REQUIRE(rc == TILEDB_OK); - REQUIRE(error == nullptr); - // Consolidate rc = tiledb_array_consolidate(ctx_, dense_vector_uri_.c_str(), config); CHECK(rc == TILEDB_OK); @@ -5414,11 +5351,6 @@ TEST_CASE_METHOD( REQUIRE(rc == TILEDB_OK); REQUIRE(error == nullptr); - rc = tiledb_config_set( - config, "sm.consolidation.buffer_size", "10000", &error); - REQUIRE(rc == TILEDB_OK); - REQUIRE(error == nullptr); - // Consolidate rc = tiledb_array_consolidate(ctx_, dense_vector_uri_.c_str(), config); CHECK(rc == TILEDB_OK); @@ -5477,11 +5409,6 @@ TEST_CASE_METHOD( REQUIRE(rc == TILEDB_OK); REQUIRE(error == nullptr); - rc = tiledb_config_set( - config, "sm.consolidation.buffer_size", "10000", &error); - REQUIRE(rc == TILEDB_OK); - REQUIRE(error == nullptr); - // Consolidate rc = tiledb_array_consolidate(ctx_, dense_vector_uri_.c_str(), config); CHECK(rc == TILEDB_OK); @@ -5540,11 +5467,6 @@ TEST_CASE_METHOD( REQUIRE(rc == TILEDB_OK); REQUIRE(error == nullptr); - rc = tiledb_config_set( - config, "sm.consolidation.buffer_size", "10000", &error); - REQUIRE(rc == TILEDB_OK); - REQUIRE(error == nullptr); - // Consolidate rc = tiledb_array_consolidate(ctx_, dense_vector_uri_.c_str(), config); CHECK(rc == TILEDB_OK); @@ -5606,11 +5528,6 @@ TEST_CASE_METHOD( REQUIRE(rc == TILEDB_OK); REQUIRE(error == nullptr); - rc = tiledb_config_set( - config, "sm.consolidation.buffer_size", "10000", &error); - REQUIRE(rc == TILEDB_OK); - REQUIRE(error == nullptr); - // Consolidate rc = tiledb_array_consolidate(ctx_, dense_vector_uri_.c_str(), config); CHECK(rc == TILEDB_OK); @@ -5677,11 +5594,6 @@ TEST_CASE_METHOD( REQUIRE(rc == TILEDB_OK); REQUIRE(error == nullptr); - rc = tiledb_config_set( - config, "sm.consolidation.buffer_size", "10000", &error); - REQUIRE(rc == TILEDB_OK); - REQUIRE(error == nullptr); - // Consolidate rc = tiledb_array_consolidate(ctx_, dense_vector_uri_.c_str(), config); CHECK(rc == TILEDB_OK); @@ -5708,66 +5620,6 @@ TEST_CASE_METHOD( remove_dense_vector(); } -TEST_CASE_METHOD( - ConsolidationFx, - "C API: Test advanced consolidation, small buffer size", - "[capi][consolidation][adv][buffer-size][non-rest]") { - remove_dense_vector(); - create_dense_vector(); - write_dense_vector_4_fragments(); - read_dense_vector(); - - tiledb_config_t* config = nullptr; - tiledb_error_t* error = nullptr; - REQUIRE(tiledb_config_alloc(&config, &error) == TILEDB_OK); - REQUIRE(error == nullptr); - - // Configure test - int rc = tiledb_config_set(config, "sm.consolidation.steps", "2", &error); - REQUIRE(rc == TILEDB_OK); - REQUIRE(error == nullptr); - rc = - tiledb_config_set(config, "sm.consolidation.step_min_frags", "2", &error); - REQUIRE(rc == TILEDB_OK); - REQUIRE(error == nullptr); - rc = - tiledb_config_set(config, "sm.consolidation.step_max_frags", "2", &error); - REQUIRE(rc == TILEDB_OK); - REQUIRE(error == nullptr); - rc = tiledb_config_set( - config, "sm.consolidation.step_size_ratio", "0.0", &error); - REQUIRE(rc == TILEDB_OK); - REQUIRE(error == nullptr); - rc = tiledb_config_set(config, "sm.consolidation.buffer_size", "10", &error); - REQUIRE(rc == TILEDB_OK); - REQUIRE(error == nullptr); - - // Consolidate - rc = tiledb_array_consolidate(ctx_, dense_vector_uri_.c_str(), config); - CHECK(rc == TILEDB_OK); - - // Vacuum - rc = tiledb_array_vacuum(ctx_, dense_vector_uri_.c_str(), nullptr); - CHECK(rc == TILEDB_OK); - - // Check correctness - read_dense_vector(); - - // Check number of fragments - get_num_struct data = {ctx_, vfs_, 0}; - rc = tiledb_vfs_ls( - ctx_, - vfs_, - vfs_test_setup_.fragment_dir(dense_vector_uri_).c_str(), - &get_dir_num, - &data); - CHECK(rc == TILEDB_OK); - CHECK(data.num == 2); - - tiledb_config_free(&config); - remove_dense_vector(); -} - TEST_CASE_METHOD( ConsolidationFx, "C API: Test advanced consolidation, encrypted array", @@ -5798,10 +5650,6 @@ TEST_CASE_METHOD( REQUIRE(rc == TILEDB_OK); REQUIRE(err == nullptr); - rc = tiledb_config_set(cfg, "sm.consolidation.buffer_size", "10000", &err); - REQUIRE(rc == TILEDB_OK); - REQUIRE(err == nullptr); - // Consolidate std::string encryption_type_string = encryption_type_str((tiledb::sm::EncryptionType)encryption_type_); @@ -5871,11 +5719,6 @@ TEST_CASE_METHOD( REQUIRE(rc == TILEDB_OK); REQUIRE(error == nullptr); - rc = tiledb_config_set( - config, "sm.consolidation.buffer_size", "10000", &error); - REQUIRE(rc == TILEDB_OK); - REQUIRE(error == nullptr); - // Consolidate rc = tiledb_array_consolidate(ctx_, dense_vector_uri_.c_str(), config); CHECK(rc == TILEDB_OK); @@ -5933,11 +5776,6 @@ TEST_CASE_METHOD( REQUIRE(rc == TILEDB_OK); REQUIRE(error == nullptr); - rc = tiledb_config_set( - config, "sm.consolidation.buffer_size", "10000", &error); - REQUIRE(rc == TILEDB_OK); - REQUIRE(error == nullptr); - // Consolidate rc = tiledb_array_consolidate(ctx_, dense_vector_uri_.c_str(), config); CHECK(rc == TILEDB_OK); @@ -6002,11 +5840,6 @@ TEST_CASE_METHOD( REQUIRE(rc == TILEDB_OK); REQUIRE(error == nullptr); - rc = tiledb_config_set( - config, "sm.consolidation.buffer_size", "10000", &error); - REQUIRE(rc == TILEDB_OK); - REQUIRE(error == nullptr); - // Consolidate rc = tiledb_array_consolidate(ctx_, dense_vector_uri_.c_str(), config); CHECK(rc == TILEDB_OK); @@ -6064,11 +5897,6 @@ TEST_CASE_METHOD( REQUIRE(rc == TILEDB_OK); REQUIRE(error == nullptr); - rc = tiledb_config_set( - config, "sm.consolidation.buffer_size", "10000", &error); - REQUIRE(rc == TILEDB_OK); - REQUIRE(error == nullptr); - // Consolidate rc = tiledb_array_consolidate(ctx_, dense_vector_uri_.c_str(), config); CHECK(rc == TILEDB_OK); @@ -6129,11 +5957,6 @@ TEST_CASE_METHOD( REQUIRE(rc == TILEDB_OK); REQUIRE(error == nullptr); - rc = tiledb_config_set( - config, "sm.consolidation.buffer_size", "10000", &error); - REQUIRE(rc == TILEDB_OK); - REQUIRE(error == nullptr); - // Consolidate rc = tiledb_array_consolidate(ctx_, dense_vector_uri_.c_str(), config); CHECK(rc == TILEDB_OK); @@ -6210,11 +6033,6 @@ TEST_CASE_METHOD( REQUIRE(error == nullptr); } - rc = tiledb_config_set( - config, "sm.consolidation.buffer_size", "10000", &error); - REQUIRE(rc == TILEDB_OK); - REQUIRE(error == nullptr); - // Consolidate rc = tiledb_array_consolidate(ctx_, dense_vector_uri_.c_str(), config); CHECK(rc == TILEDB_OK); @@ -6271,10 +6089,6 @@ TEST_CASE_METHOD( REQUIRE(rc == TILEDB_OK); REQUIRE(err == nullptr); - rc = tiledb_config_set(cfg, "sm.consolidation.buffer_size", "10000", &err); - REQUIRE(rc == TILEDB_OK); - REQUIRE(err == nullptr); - // Consolidate rc = tiledb_array_consolidate(ctx_, dense_vector_uri_.c_str(), cfg); CHECK(rc == TILEDB_OK); @@ -6564,7 +6378,7 @@ TEST_CASE_METHOD( TEST_CASE_METHOD( ConsolidationFx, "C API: Test consolidation, sparse string", - "[capi][consolidation][sparse][string][non-rest]") { + "[!mayfail][capi][consolidation][sparse][string][non-rest]") { remove_sparse_string_array(); create_sparse_string_array(); @@ -7279,32 +7093,19 @@ TEST_CASE_METHOD( rc = tiledb_fragment_info_get_fragment_uri(ctx_, fragment_info, 3, &uri2); CHECK(rc == TILEDB_OK); - // Set consolidation buffer size - tiledb_config_t* cfg; - tiledb_error_t* err = nullptr; - - rc = tiledb_config_alloc(&cfg, &err); - REQUIRE(rc == TILEDB_OK); - REQUIRE(err == nullptr); - - rc = tiledb_config_set(cfg, "sm.consolidation.buffer_size", "10000", &err); - REQUIRE(rc == TILEDB_OK); - REQUIRE(err == nullptr); - // Consolidate SECTION("Relative URIs") { const char* uris[2] = {strrchr(uri, '/') + 1, strrchr(uri2, '/') + 1}; rc = tiledb_array_consolidate_fragments( - ctx_, dense_array_uri_.c_str(), uris, 2, cfg); + ctx_, dense_array_uri_.c_str(), uris, 2, nullptr); } SECTION("Absolute URIs") { const char* uris[2] = {uri, uri2}; rc = tiledb_array_consolidate_fragments( - ctx_, dense_array_uri_.c_str(), uris, 2, cfg); + ctx_, dense_array_uri_.c_str(), uris, 2, nullptr); } CHECK(rc == TILEDB_OK); - tiledb_config_free(&cfg); tiledb_fragment_info_free(&fragment_info); @@ -7371,29 +7172,17 @@ TEST_CASE_METHOD( rc = tiledb_fragment_info_get_fragment_uri(ctx_, fragment_info, 3, &uri2); CHECK(rc == TILEDB_OK); - // Set consolidation buffer size - tiledb_config_t* cfg; - tiledb_error_t* err = nullptr; - - rc = tiledb_config_alloc(&cfg, &err); - REQUIRE(rc == TILEDB_OK); - REQUIRE(err == nullptr); - - rc = tiledb_config_set(cfg, "sm.consolidation.buffer_size", "10000", &err); - REQUIRE(rc == TILEDB_OK); - REQUIRE(err == nullptr); - // Consolidate SECTION("Relative URIs") { const char* uris[2] = {strrchr(uri, '/') + 1, strrchr(uri2, '/') + 1}; rc = tiledb_array_consolidate_fragments( - ctx_, sparse_array_uri_.c_str(), uris, 2, cfg); + ctx_, sparse_array_uri_.c_str(), uris, 2, nullptr); } SECTION("Absolute URIs") { const char* uris[2] = {uri, uri2}; rc = tiledb_array_consolidate_fragments( - ctx_, sparse_array_uri_.c_str(), uris, 2, cfg); + ctx_, sparse_array_uri_.c_str(), uris, 2, nullptr); } SECTION("Invalid URIs") { @@ -7402,12 +7191,11 @@ TEST_CASE_METHOD( frag2 = "/some/array/__fragments/" + frag2; const char* uris[2] = {frag1.c_str(), frag2.c_str()}; rc = tiledb_array_consolidate_fragments( - ctx_, sparse_array_uri_.c_str(), uris, 2, cfg); + ctx_, sparse_array_uri_.c_str(), uris, 2, nullptr); CHECK(rc == TILEDB_ERR); return; } CHECK(rc == TILEDB_OK); - tiledb_config_free(&cfg); tiledb_fragment_info_free(&fragment_info); @@ -7482,29 +7270,17 @@ TEST_CASE_METHOD( rc = tiledb_fragment_info_get_fragment_uri(ctx_, fragment_info, 3, &uri2); CHECK(rc == TILEDB_OK); - // Set consolidation buffer size - tiledb_config_t* cfg; - tiledb_error_t* err = nullptr; - - rc = tiledb_config_alloc(&cfg, &err); - REQUIRE(rc == TILEDB_OK); - REQUIRE(err == nullptr); - - rc = tiledb_config_set(cfg, "sm.consolidation.buffer_size", "10000", &err); - REQUIRE(rc == TILEDB_OK); - REQUIRE(err == nullptr); - // Consolidate SECTION("Relative URIs") { const char* uris[2] = {strrchr(uri, '/') + 1, strrchr(uri2, '/') + 1}; rc = tiledb_array_consolidate_fragments( - ctx_, sparse_array_uri_.c_str(), uris, 2, cfg); + ctx_, sparse_array_uri_.c_str(), uris, 2, nullptr); } SECTION("Absolute URIs") { const char* uris[2] = {uri, uri2}; rc = tiledb_array_consolidate_fragments( - ctx_, sparse_array_uri_.c_str(), uris, 2, cfg); + ctx_, sparse_array_uri_.c_str(), uris, 2, nullptr); } SECTION("Invalid URIs") { @@ -7513,12 +7289,11 @@ TEST_CASE_METHOD( frag2 = "/some/array/" + frag2; const char* uris[2] = {frag1.c_str(), frag2.c_str()}; rc = tiledb_array_consolidate_fragments( - ctx_, sparse_array_uri_.c_str(), uris, 2, cfg); + ctx_, sparse_array_uri_.c_str(), uris, 2, nullptr); CHECK(rc == TILEDB_ERR); return; } CHECK(rc == TILEDB_OK); - tiledb_config_free(&cfg); tiledb_fragment_info_free(&fragment_info); @@ -7590,13 +7365,13 @@ TEST_CASE_METHOD( TEST_CASE_METHOD( ConsolidationFx, "C API: Test consolidation, sparse string, no progress", - "[capi][consolidation][sparse][string][no-progress][non-rest]") { + "[!mayfail][capi][consolidation][sparse][string][no-progress][non-rest]") { remove_sparse_string_array(); create_sparse_string_array(); write_sparse_string_full(); write_sparse_string_unordered(); - consolidate_sparse_string(1, true); + consolidate_sparse_string(true); tiledb_error_t* err = NULL; tiledb_ctx_get_last_error(ctx_, &err); diff --git a/test/src/unit-capi-string_dims.cc b/test/src/unit-capi-string_dims.cc index 9bd483754c9..0c1047e4700 100644 --- a/test/src/unit-capi-string_dims.cc +++ b/test/src/unit-capi-string_dims.cc @@ -5,7 +5,7 @@ * * The MIT License * - * @copyright Copyright (c) 2017-2024 TileDB Inc. + * @copyright Copyright (c) 2017-2026 TileDB Inc. * * Permission is hereby granted, free of charge, to any person obtaining a copy * of this software and associated documentation files (the "Software"), to deal @@ -1680,21 +1680,10 @@ TEST_CASE_METHOD( CHECK(rc == TILEDB_OK); CHECK(dirs.num == 2); - tiledb_config_t* config = nullptr; - tiledb_error_t* error = nullptr; - REQUIRE(tiledb_config_alloc(&config, &error) == TILEDB_OK); - REQUIRE(error == nullptr); - - rc = tiledb_config_set( - config, "sm.consolidation.buffer_size", "10000", &error); - REQUIRE(rc == TILEDB_OK); - REQUIRE(error == nullptr); - // Consolidate - rc = tiledb_array_consolidate(ctx_, array_name.c_str(), config); + rc = tiledb_array_consolidate(ctx_, array_name.c_str(), nullptr); CHECK(rc == TILEDB_OK); rc = tiledb_array_vacuum(ctx_, array_name.c_str(), nullptr); - tiledb_config_free(&config); // Check number of fragments dirs = {ctx_, vfs_, 0}; diff --git a/test/src/unit-cppapi-consolidation-sparse.cc b/test/src/unit-cppapi-consolidation-sparse.cc index f1b60494f78..46378c2ba4b 100644 --- a/test/src/unit-cppapi-consolidation-sparse.cc +++ b/test/src/unit-cppapi-consolidation-sparse.cc @@ -5,7 +5,7 @@ * * The MIT License * - * @copyright Copyright (c) 2017-2021 TileDB Inc. + * @copyright Copyright (c) 2017-2026 TileDB Inc. * * Permission is hereby granted, free of charge, to any person obtaining a copy * of this software and associated documentation files (the "Software"), to deal @@ -108,11 +108,9 @@ TEST_CASE( read_array(array_name, {1, 2, 3}, {1, 2, 3}); Context ctx; - Config config; - config["sm.consolidation.buffer_size"] = "8"; - REQUIRE_NOTHROW(Array::consolidate(ctx, array_name, &config)); + REQUIRE_NOTHROW(Array::consolidate(ctx, array_name, nullptr)); CHECK(tiledb::test::num_fragments(array_name) == 3); - REQUIRE_NOTHROW(Array::vacuum(ctx, array_name, &config)); + REQUIRE_NOTHROW(Array::vacuum(ctx, array_name, nullptr)); CHECK(tiledb::test::num_fragments(array_name) == 1); read_array(array_name, {1, 2, 3}, {1, 2, 3}); @@ -142,9 +140,7 @@ TEST_CASE( read_array(array_name, {1, 2, 3}, {1, 2, 3}); Context ctx; - Config config; - config["sm.consolidation.buffer_size"] = "8"; - REQUIRE_NOTHROW(Array::consolidate(ctx, array_name, &config)); + REQUIRE_NOTHROW(Array::consolidate(ctx, array_name, nullptr)); CHECK(tiledb::test::num_fragments(array_name) == 3); read_array(array_name, {1, 2, 3}, {1, 2, 3}); diff --git a/test/src/unit-cppapi-consolidation-with-timestamps.cc b/test/src/unit-cppapi-consolidation-with-timestamps.cc index f3960fe8d1e..1d21fef8a04 100644 --- a/test/src/unit-cppapi-consolidation-with-timestamps.cc +++ b/test/src/unit-cppapi-consolidation-with-timestamps.cc @@ -5,7 +5,7 @@ * * The MIT License * - * @copyright Copyright (c) 2023 TileDB Inc. + * @copyright Copyright (c) 2023-2026 TileDB Inc. * * Permission is hereby granted, free of charge, to any person obtaining a copy * of this software and associated documentation files (the "Software"), to deal @@ -94,11 +94,6 @@ struct ConsolidationWithTimestampsFx { ConsolidationWithTimestampsFx::ConsolidationWithTimestampsFx() : vfs_(ctx_) { - Config config; - config.set("sm.consolidation.buffer_size", "1000"); - ctx_ = Context(config); - resources_ = &ctx_.ptr().get()->resources(); - vfs_ = VFS(ctx_); } ConsolidationWithTimestampsFx::~ConsolidationWithTimestampsFx() { @@ -106,10 +101,8 @@ ConsolidationWithTimestampsFx::~ConsolidationWithTimestampsFx() { void ConsolidationWithTimestampsFx::set_legacy() { Config config; - config.set("sm.consolidation.buffer_size", "1000"); config.set("sm.query.sparse_global_order.reader", "legacy"); config.set("sm.query.sparse_unordered_with_dups.reader", "legacy"); - ctx_ = Context(config); resources_ = &ctx_.ptr().get()->resources(); vfs_ = VFS(ctx_); diff --git a/test/src/unit-cppapi-consolidation.cc b/test/src/unit-cppapi-consolidation.cc index 93b92eecc73..82c5a0247af 100644 --- a/test/src/unit-cppapi-consolidation.cc +++ b/test/src/unit-cppapi-consolidation.cc @@ -5,7 +5,7 @@ * * The MIT License * - * @copyright Copyright (c) 2017-2021 TileDB Inc. + * @copyright Copyright (c) 2017-2026 TileDB Inc. * * Permission is hereby granted, free of charge, to any person obtaining a copy * of this software and associated documentation files (the "Software"), to deal @@ -126,11 +126,9 @@ TEST_CASE( read_array(array_name, {1, 3}, {1, 2, 3}); Context ctx; - Config config; - config["sm.consolidation.buffer_size"] = "4"; - REQUIRE_NOTHROW(Array::consolidate(ctx, array_name, &config)); + REQUIRE_NOTHROW(Array::consolidate(ctx, array_name, nullptr)); CHECK(tiledb::test::num_fragments(array_name) == 3); - REQUIRE_NOTHROW(Array::vacuum(ctx, array_name, &config)); + REQUIRE_NOTHROW(Array::vacuum(ctx, array_name, nullptr)); CHECK(tiledb::test::num_fragments(array_name) == 1); read_array(array_name, {1, 3}, {1, 2, 3}); @@ -222,9 +220,7 @@ TEST_CASE( read_array(array_name, {1, 3}, {1, 2, 3}); Context ctx; - Config config; - config["sm.consolidation.buffer_size"] = "4"; - REQUIRE_NOTHROW(Array::consolidate(ctx, array_name, &config)); + REQUIRE_NOTHROW(Array::consolidate(ctx, array_name, nullptr)); CHECK(tiledb::test::num_fragments(array_name) == 3); read_array(array_name, {1, 3}, {1, 2, 3}); @@ -246,8 +242,6 @@ TEST_CASE( read_array(array_name, {1, 3}, {1, 2, 3}); Context ctx; - Config config; - config.set("sm.consolidation.buffer_size", "1000"); FragmentInfo fragment_info(ctx, array_name); fragment_info.load(); @@ -262,7 +256,7 @@ TEST_CASE( short_fragment_name1.c_str(), short_fragment_name2.c_str()}; REQUIRE_NOTHROW( - Array::consolidate(ctx, array_name, fragment_uris, 2, &config)); + Array::consolidate(ctx, array_name, fragment_uris, 2, nullptr)); CHECK(tiledb::test::num_fragments(array_name) == 3); read_array(array_name, {1, 3}, {1, 2, 3}); @@ -422,10 +416,7 @@ TEST_CASE( TEST_CASE( "C++ API: Test consolidation with timestamp and max domain", "[cppapi][consolidation][timestamp][maxdomain]") { - Config cfg; - cfg["sm.consolidation.buffer_size"] = "10000"; - - Context ctx(cfg); + Context ctx; VFS vfs(ctx); const std::string array_name = "consolidate_timestamp_max_domain"; diff --git a/test/src/unit-cppapi-deletes.cc b/test/src/unit-cppapi-deletes.cc index a565000049b..5899f666396 100644 --- a/test/src/unit-cppapi-deletes.cc +++ b/test/src/unit-cppapi-deletes.cc @@ -5,7 +5,7 @@ * * The MIT License * - * @copyright Copyright (c) 2023-2024 TileDB Inc. + * @copyright Copyright (c) 2023-2026 TileDB Inc. * * Permission is hereby granted, free of charge, to any person obtaining a copy * of this software and associated documentation files (the "Software"), to deal @@ -120,16 +120,12 @@ DeletesFx::DeletesFx() , array_name_(vfs_test_setup_.array_uri(SPARSE_ARRAY_NAME)) , vfs_array_name_(vfs_test_setup_.array_uri(SPARSE_ARRAY_NAME, true)) , group_name_(vfs_test_setup_.array_uri(GROUP_NAME)) { - Config config; - config.set("sm.consolidation.buffer_size", "1000"); - vfs_test_setup_.update_config(config.ptr().get()); ctx_ = vfs_test_setup_.ctx(); vfs_ = VFS(ctx_); } void DeletesFx::set_purge_deleted_cells() { Config config; - config.set("sm.consolidation.buffer_size", "1000"); config.set("sm.consolidation.purge_deleted_cells", "true"); vfs_test_setup_.update_config(config.ptr().get()); ctx_ = vfs_test_setup_.ctx(); @@ -138,7 +134,6 @@ void DeletesFx::set_purge_deleted_cells() { void DeletesFx::set_legacy() { Config config; - config.set("sm.consolidation.buffer_size", "1000"); config.set("sm.query.sparse_global_order.reader", "legacy"); config.set("sm.query.sparse_unordered_with_dups.reader", "legacy"); vfs_test_setup_.update_config(config.ptr().get()); @@ -170,14 +165,13 @@ void DeletesFx::create_simple_array(const std::string& path) { } void DeletesFx::create_sparse_array(bool allows_dups, bool encrypt) { - Config config; - config.set("sm.consolidation.buffer_size", "1000"); if (encrypt) { + Config config; config["sm.encryption_type"] = enc_type_str_.c_str(); config["sm.encryption_key"] = key_; + vfs_test_setup_.update_config(config.ptr().get()); } - vfs_test_setup_.update_config(config.ptr().get()); ctx_ = vfs_test_setup_.ctx(); vfs_ = VFS(ctx_); diff --git a/test/src/unit-cppapi-hilbert.cc b/test/src/unit-cppapi-hilbert.cc index 2028646b6f7..5ed9f3a466f 100644 --- a/test/src/unit-cppapi-hilbert.cc +++ b/test/src/unit-cppapi-hilbert.cc @@ -5,7 +5,7 @@ * * The MIT License * - * @copyright Copyright (c) 2017-2023 TileDB Inc. + * @copyright Copyright (c) 2017-2026 TileDB Inc. * * Permission is hereby granted, free of charge, to any person obtaining a copy * of this software and associated documentation files (the "Software"), to deal @@ -701,10 +701,7 @@ TEST_CASE( TEST_CASE( "C++ API: Test Hilbert, consolidation", "[cppapi][hilbert][consolidation][non-rest]") { - Config cfg; - cfg["sm.consolidation.buffer_size"] = "10000"; - - Context ctx(cfg); + Context ctx; VFS vfs(ctx); std::string array_name = "hilbert_array"; @@ -1053,10 +1050,7 @@ TEST_CASE( TEST_CASE( "C++ API: Test Hilbert, 2d, int32, negative, consolidation", "[cppapi][hilbert][2d][int32][negative][consolidation]") { - Config cfg; - cfg["sm.consolidation.buffer_size"] = "10000"; - - Context ctx(cfg); + Context ctx; VFS vfs(ctx); std::string array_name = "hilbert_array"; @@ -1460,10 +1454,7 @@ TEST_CASE( TEST_CASE( "C++ API: Test Hilbert, 2d, float32, consolidation", "[cppapi][hilbert][2d][float32][consolidation][non-rest]") { - Config cfg; - cfg["sm.consolidation.buffer_size"] = "10000"; - - Context ctx(cfg); + Context ctx; VFS vfs(ctx); std::string array_name = "hilbert_array"; diff --git a/test/src/unit-cppapi-metadata.cc b/test/src/unit-cppapi-metadata.cc index ae237a0f6d1..d627981e25a 100644 --- a/test/src/unit-cppapi-metadata.cc +++ b/test/src/unit-cppapi-metadata.cc @@ -5,7 +5,7 @@ * * The MIT License * - * @copyright Copyright (c) 2017-2021 TileDB, Inc. + * @copyright Copyright (c) 2017-2026 TileDB, Inc. * * Permission is hereby granted, free of charge, to any person obtaining a copy * of this software and associated documentation files (the "Software"), to deal @@ -406,14 +406,11 @@ TEST_CASE_METHOD( CPPMetadataFx, "C++ API: Metadata, multiple metadata and consolidate", "[cppapi][metadata][multiple][consolidation]") { - Config cfg; - cfg["sm.consolidation.buffer_size"] = "10000"; - // Create default array create_default_array_1d(); // Create and open array in write mode - tiledb::Context ctx(cfg); + tiledb::Context ctx; tiledb::Array array(ctx, array_name_, TILEDB_WRITE); // Write items diff --git a/test/src/unit-cppapi-update-queries.cc b/test/src/unit-cppapi-update-queries.cc index 0a60ca03007..dfb9bbdbaa4 100644 --- a/test/src/unit-cppapi-update-queries.cc +++ b/test/src/unit-cppapi-update-queries.cc @@ -5,7 +5,7 @@ * * The MIT License * - * @copyright Copyright (c) 2017-2024 TileDB Inc. + * @copyright Copyright (c) 2017-2026 TileDB Inc. * * Permission is hereby granted, free of charge, to any person obtaining a copy * of this software and associated documentation files (the "Software"), to deal @@ -81,7 +81,6 @@ struct UpdatesFx { UpdatesFx::UpdatesFx() : vfs_(ctx_) { Config config; - config.set("sm.consolidation.buffer_size", "1000"); config["sm.allow_updates_experimental"] = "true"; ctx_ = Context(config); vfs_ = VFS(ctx_); diff --git a/tiledb/api/c_api/config/config_api_external.h b/tiledb/api/c_api/config/config_api_external.h index 0492b28edfc..cc68911afbb 100644 --- a/tiledb/api/c_api/config/config_api_external.h +++ b/tiledb/api/c_api/config/config_api_external.h @@ -5,7 +5,7 @@ * * The MIT License * - * @copyright Copyright (c) 2023-2025 TileDB, Inc. + * @copyright Copyright (c) 2023-2026 TileDB, Inc. * * Permission is hereby granted, free of charge, to any person obtaining a copy * of this software and associated documentation files (the "Software"), to deal @@ -166,11 +166,6 @@ TILEDB_EXPORT void tiledb_config_free(tiledb_config_t** config) TILEDB_NOEXCEPT; * fragment will have to fill with the special fill value * (since the resulting fragment is dense).
* **Default**: 1.0 - * - `sm.consolidation.buffer_size`
- * **Deprecated** - * The size (in bytes) of the attribute buffers used during - * consolidation.
- * **Default**: 50,000,000 * - `sm.consolidation.max_fragment_size`
* **Experimental**
* The size (in bytes) of the maximum on-disk fragment size that will be diff --git a/tiledb/sm/config/config.cc b/tiledb/sm/config/config.cc index 3086b49aa1f..e5b3eb01449 100644 --- a/tiledb/sm/config/config.cc +++ b/tiledb/sm/config/config.cc @@ -5,7 +5,7 @@ * * The MIT License * - * @copyright Copyright (c) 2017-2025 TileDB, Inc. + * @copyright Copyright (c) 2017-2026 TileDB, Inc. * * Permission is hereby granted, free of charge, to any person obtaining a copy * of this software and associated documentation files (the "Software"), to deal @@ -355,8 +355,6 @@ const std::map default_config_values = { std::make_pair( "sm.consolidation.amplification", Config::SM_CONSOLIDATION_AMPLIFICATION), - std::make_pair( - "sm.consolidation.buffer_size", Config::SM_CONSOLIDATION_BUFFER_SIZE), std::make_pair( "sm.consolidation.max_fragment_size", Config::SM_CONSOLIDATION_MAX_FRAGMENT_SIZE), @@ -797,8 +795,6 @@ Status Config::sanity_check( RETURN_NOT_OK(utils::parse::convert(value, &vuint64)); } else if (param == "sm.consolidation.amplification") { RETURN_NOT_OK(utils::parse::convert(value, &vf)); - } else if (param == "sm.consolidation.buffer_size") { - RETURN_NOT_OK(utils::parse::convert(value, &vuint64)); } else if (param == "sm.consolidation.max_fragment_size") { RETURN_NOT_OK(utils::parse::convert(value, &vuint64)); } else if (param == "sm.consolidation.purge_deleted_cells") { diff --git a/tiledb/sm/consolidator/fragment_consolidator.cc b/tiledb/sm/consolidator/fragment_consolidator.cc index db6f9bcad4d..79cd7eb104c 100644 --- a/tiledb/sm/consolidator/fragment_consolidator.cc +++ b/tiledb/sm/consolidator/fragment_consolidator.cc @@ -135,13 +135,8 @@ void FragmentConsolidationWorkspace::resize_buffers( buffer_weights.emplace_back(sizeof(uint64_t)); } - // If a user set the per-attribute buffer size configuration, we override - // the use of the total_budget_size config setting for backwards - // compatible behavior. auto buffer_num = buffer_weights.size(); - if (config.buffer_size_ != 0) { - total_buffers_budget = config.buffer_size_ * buffer_num; - } + total_buffers_budget = buffer_num; // Calculate the size of individual buffers by assigning a weight based // percentage of the total buffer size. @@ -1009,17 +1004,6 @@ Status FragmentConsolidator::set_config(const Config& config) { "sm.consolidation.amplification", Config::must_find); config_.steps_ = merged_config.get("sm.consolidation.steps", Config::must_find); - config_.buffer_size_ = 0; - // Only set the buffer_size_ if the user specified a value. Otherwise, we use - // the new sm.mem.consolidation.buffers_weight instead. - if (merged_config.set_params().count("sm.consolidation.buffer_size") > 0) { - logger_->warn( - "The `sm.consolidation.buffer_size configuration setting has been " - "deprecated. Set consolidation buffer sizes using the newer " - "`sm.mem.consolidation.buffers_weight` setting."); - config_.buffer_size_ = merged_config.get( - "sm.consolidation.buffer_size", Config::must_find); - } config_.total_budget_ = merged_config.get("sm.mem.total_budget", Config::must_find); config_.buffers_weight_ = merged_config.get( diff --git a/tiledb/sm/consolidator/fragment_consolidator.h b/tiledb/sm/consolidator/fragment_consolidator.h index 3fcae353d7e..c7157ac6d32 100644 --- a/tiledb/sm/consolidator/fragment_consolidator.h +++ b/tiledb/sm/consolidator/fragment_consolidator.h @@ -5,7 +5,7 @@ * * The MIT License * - * @copyright Copyright (c) 2022-2024 TileDB, Inc. + * @copyright Copyright (c) 2022-2026 TileDB, Inc. * * Permission is hereby granted, free of charge, to any person obtaining a copy * of this software and associated documentation files (the "Software"), to deal @@ -78,8 +78,6 @@ struct FragmentConsolidationConfig : Consolidator::ConsolidationConfigBase { * (since the resulting fragments is dense). */ float amplification_; - /** Attribute buffer size. */ - uint64_t buffer_size_; /** Total memory budget for consolidation operation. */ uint64_t total_budget_; /** Consolidation buffers weight used to partition total budget. */ diff --git a/tiledb/sm/consolidator/test/unit_fragment_consolidator.cc b/tiledb/sm/consolidator/test/unit_fragment_consolidator.cc index f67412aa495..de96a5265f7 100644 --- a/tiledb/sm/consolidator/test/unit_fragment_consolidator.cc +++ b/tiledb/sm/consolidator/test/unit_fragment_consolidator.cc @@ -5,7 +5,7 @@ * * The MIT License * - * @copyright Copyright (c) 2022-2024 TileDB, Inc. + * @copyright Copyright (c) 2022-2026 TileDB, Inc. * * Permission is hereby granted, free of charge, to any person obtaining a copy * of this software and associated documentation files (the "Software"), to deal @@ -220,7 +220,6 @@ TEST_CASE( FragmentConsolidationConfig cfg; cfg.with_timestamps_ = with_timestamps; cfg.with_delete_meta_ = with_delete_meta; - cfg.buffer_size_ = 1000; FragmentConsolidationWorkspace cw(tiledb::test::get_test_memory_tracker()); cw.resize_buffers(&statistics, cfg, *schema, avg_cell_sizes, 1); diff --git a/tiledb/sm/cpp_api/config.h b/tiledb/sm/cpp_api/config.h index cb2a9726461..ae5ca75a94c 100644 --- a/tiledb/sm/cpp_api/config.h +++ b/tiledb/sm/cpp_api/config.h @@ -5,7 +5,7 @@ * * The MIT License * - * @copyright Copyright (c) 2017-2025 TileDB, Inc. + * @copyright Copyright (c) 2017-2026 TileDB, Inc. * * Permission is hereby granted, free of charge, to any person obtaining a copy * of this software and associated documentation files (the "Software"), to deal @@ -337,11 +337,6 @@ class Config { * fragment will have to fill with the special fill value * (since the resulting fragment is dense).
* **Default**: 1.0 - * - `sm.consolidation.buffer_size`
- * **Deprecated** - * The size (in bytes) of the attribute buffers used during - * consolidation.
- * **Default**: 50,000,000 * - `sm.consolidation.max_fragment_size`
* **Experimental**
* The size (in bytes) of the maximum on-disk fragment size that will be