-
Notifications
You must be signed in to change notification settings - Fork 178
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
Add api for getting array schema of a fragment #2468
Conversation
This pull request has been linked to Clubhouse Story #9809: Add c/cpp api for getting array schema of a fragment. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One minor change needed in the c-api
.
} | ||
|
||
EncryptionKey encryption_key; | ||
RETURN_NOT_OK(storage_manager_->load_array_schema_from_uri( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is a deficiency here in that we'll be loading the schema for every fragment without any caching. I realize we don't have the openarray
object here so we can't use the normal schema map. Lets open a a story for a future improvement here but it doesn't block this PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* add api for getting array schema of a fragment * delete array_schema pointer when error is caught
This PR adds c and cpp api functions for getting the array schema of a fragment from fragment info object. We store array schema name in the meta data of each fragment. Now we can get the array schema in capi by the tiledb_fragment_info_get_array_schema function. Also a member function array_schema is added to FragmentInfo class in cpp-api.
TYPE: FEATURE
DESC: Add c and cpp api functions for getting the array schema of a fragment
TYPE: C_API
DESC: Add
tiledb_fragment_info_get_array_schema
functions for getting the array schema of a fragmentTYPE: CPP_API
DESC: Add
FragmentInfo::array_schema
functions for getting the array schema of a fragment