Skip to content

Commit 9ad883c

Browse files
author
Nikos Papailiou
committed
Fix test
1 parent 0a08b85 commit 9ad883c

File tree

2 files changed

+8
-9
lines changed

2 files changed

+8
-9
lines changed

apis/python/src/tiledb/vector_search/ingestion.py

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1743,14 +1743,12 @@ def consolidate_and_vacuum(
17431743
conf = tiledb.Config(config)
17441744
conf["sm.consolidation.mode"] = mode
17451745
conf["sm.vacuum.mode"] = mode
1746-
# TODO consolidate remote arrays when this is supported in tiledb API
1747-
if not index_group_uri.startswith("tiledb://"):
1748-
ids_uri = group[IDS_ARRAY_NAME].uri
1749-
parts_uri = group[PARTS_ARRAY_NAME].uri
1750-
tiledb.consolidate(parts_uri, config=conf)
1751-
tiledb.vacuum(parts_uri, config=conf)
1752-
tiledb.consolidate(ids_uri, config=conf)
1753-
tiledb.vacuum(ids_uri, config=conf)
1746+
ids_uri = group[IDS_ARRAY_NAME].uri
1747+
parts_uri = group[PARTS_ARRAY_NAME].uri
1748+
tiledb.consolidate(parts_uri, config=conf)
1749+
tiledb.vacuum(parts_uri, config=conf)
1750+
tiledb.consolidate(ids_uri, config=conf)
1751+
tiledb.vacuum(ids_uri, config=conf)
17541752

17551753
# TODO remove temp data for tiledb URIs
17561754
if not index_group_uri.startswith("tiledb://"):

apis/python/test/test_cloud.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,10 +13,11 @@
1313
class CloudTests(unittest.TestCase):
1414
def setUp(self):
1515
super().setUp()
16+
tiledb.cloud.login(token=os.getenv("TILEDB_REST_TOKEN"))
1617
self.namespace, storage_path, _ = groups._default_ns_path_cred()
18+
storage_path = storage_path.replace("//", "/").replace("/", "//", 1)
1719
rand_name = random_name("vector_search")
1820
self.test_path = f"tiledb://{self.namespace}/{storage_path}/{rand_name}"
19-
tiledb.cloud.login(token=os.getenv("TILEDB_REST_TOKEN"))
2021
self.config = tiledb.cloud.Config()
2122
self.config_dict = self.config.dict()
2223

0 commit comments

Comments
 (0)