Remove deprecated dump APIs
#5585
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
teo-tsirpanis
approved these changes
Jul 16, 2025
Member
|
REST-CI fails because TileDB-Go references the now-deleted |
teo-tsirpanis
requested changes
Jul 16, 2025
Member
teo-tsirpanis
left a comment
teo-tsirpanis
left a comment
There was a problem hiding this comment.
Reverting approval until we migrate TileDB-Go.
Member
Author
|
TileDB-Go PR: TileDB-Inc/TileDB-Go#378 |
bekadavis9
approved these changes
Jul 18, 2025
Contributor
bekadavis9
left a comment
bekadavis9
left a comment
There was a problem hiding this comment.
Thanks for getting this done so quickly!
Member
Author
|
@teo-tsirpanis, you are blocking merging because you had requested for changes. |
teo-tsirpanis
approved these changes
Nov 18, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Following #5026 and #5179, and in accordance with our deprecation cycle, several dump C APIs and their corresponding CPP APIs have been deprecated for three releases - since version 2.26 - and should be removed with the release 2.29. This PR removes them. The C APIs removed are:
tiledb_array_schema_dumptiledb_enumeration_dumptiledb_attribute_dumptiledb_dimension_dumptiledb_domain_dumptiledb_group_dump_strGoing forward, the corresponding
*_dump_strC APIs and the C++operator<<overload should be used instead. For the removedtiledb_group_dump_str, thetiledb_group_dump_str_v2C API should be used as a replacement.Closes CORE-302
TYPE: DEPRECATION
DESC: Remove deprecated dump APIs