Skip to content

Commit 36da774

Browse files
authored
Do not provide storage URIs in TileDB-Server REST requests. (#5687)
Support for custom storage paths was removed in TileDB-Server, causing REST CI failures since core sets the URI field for array / group creation requests. This updates the requests to not set the URI field if we are talking to a 3.0 REST server. --- TYPE: BUG DESC: Do not provide storage URIs in TileDB-Server REST requests.
1 parent 9bfedba commit 36da774

File tree

6 files changed

+238
-77
lines changed

6 files changed

+238
-77
lines changed

test/src/unit-capi-array.cc

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,7 @@
3939
#include "test/support/src/helpers.h"
4040
#include "test/support/src/serialization_wrappers.h"
4141
#include "test/support/src/vfs_helpers.h"
42+
#include "tiledb/api/c_api/vfs/vfs_api_internal.h"
4243
#ifdef _WIN32
4344
#include "tiledb/sm/filesystem/win.h"
4445
#else
@@ -85,6 +86,9 @@ struct ArrayFx {
8586
// The memory tracker
8687
shared_ptr<tiledb::sm::MemoryTracker> memory_tracker_;
8788

89+
// TODO: Update ArrayFx to use VFSTestSetup.
90+
VFSTestSetup vfs_test_setup_;
91+
8892
// TileDB context
8993
tiledb_ctx_t* ctx_;
9094
tiledb_vfs_t* vfs_;
@@ -901,6 +905,7 @@ TEST_CASE_METHOD(
901905
create_temp_dir(temp_dir);
902906

903907
create_dense_vector(array_name);
908+
array_path = vfs_test_setup_.get_backend_uri(array_name);
904909

905910
// ---- FIRST WRITE ----
906911
// Prepare cell buffers
@@ -1934,6 +1939,7 @@ TEST_CASE_METHOD(
19341939
create_temp_dir(temp_dir);
19351940

19361941
create_dense_vector(array_name);
1942+
array_path = vfs_test_setup_.get_backend_uri(array_name);
19371943

19381944
// Conditionally consolidate
19391945
// Note: there's no need to vacuum; delete_array will delete all fragments

0 commit comments

Comments
 (0)