TileDB icon indicating copy to clipboard operation
TileDB copied to clipboard

Consolidation Plan REST support - part 2: rest client

Open ypatia opened this issue 2 years ago • 3 comments

This is part2 of https://github.com/TileDB-Inc/TileDB/pull/4534 , adding the rest_client part. The REST route needs to be added on the Cloud side (the actual route needs confirmation/specification from the Cloud team before this can be merged).


TYPE: IMPROVEMENT DESC: Consolidation Plan REST support - part 2: rest client

ypatia avatar Nov 24 '23 11:11 ypatia

I should mention, other than that, the existing changes look perfectly fine.

davisp avatar Nov 29 '23 20:11 davisp

Lets make this a draft to avoid an accidental merge.

We should also add tests for the REST CI runner. Those are conditionally enabled here:

https://github.com/TileDB-Inc/TileDB/blob/dev/test/CMakeLists.txt#L262-L267

Moved to Draft and added some testing for REST CI. I actually parametrized the existing test suite for c api consolidation plan, let me know if that works. I can't really test until Cloud changes are in..

ypatia avatar Nov 30 '23 10:11 ypatia