Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[pull] dev from TileDB-Inc:dev #160

Merged
merged 2 commits into from
Jun 8, 2024
Merged

[pull] dev from TileDB-Inc:dev #160

merged 2 commits into from
Jun 8, 2024

Conversation

pull[bot]
Copy link

@pull pull bot commented Jun 8, 2024

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

bekadavis9 and others added 2 commits June 7, 2024 10:23
…ut of StorageManager. (#5034)

Remove `Status_StorageManagerError` and its occurrences.
Related subsequent fixes:
* Remove `Logger::status_no_return_value` and replace occurrences with
`Logger::error`.
* Remove `Status` and `std::optional` from returned tuple of
`Array::open_for_writes`.
* de-`Status` `Context::init_loggers`.
* Migrate `StorageManagerCanonical::group_metadata_vacuum` -> `static
Group::vacuum_metadata`.

[sc-48630]
[sc-48639]

---
TYPE: NO_HISTORY
DESC: Remove `Status_StorageManagerError` and Migrate
`group_metadata_vacuum` out of `StorageManager`.
This PR adds the foundation for the CurrentDomain API:
- format change in array schema and disk serialization
- plumbing so you can create/open an array with current_domain data
using sm APIs
- array schema dump extension
- test coverage

To be done C and CPP API that wrap APIs defined here.

[sc-42489]

---
TYPE: FEATURE
DESC: Add CurrentDomain API support.

---------

Co-authored-by: Luc Rancourt <[email protected]>
Co-authored-by: KiterLuc <[email protected]>
@pull pull bot added the ⤵️ pull label Jun 8, 2024
@kokizzu kokizzu merged commit 9116d3c into kokizzu:dev Jun 8, 2024
55 of 61 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants