-
Notifications
You must be signed in to change notification settings - Fork 185
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
Remove Array::get_max_buffer_sizes
.
#5292
Conversation
Rest PR at: https://github.com/TileDB-Inc/TileDB-Cloud-REST/pull/4852 |
c857b28
to
f8c6a2e
Compare
This comment was marked as outdated.
This comment was marked as outdated.
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.
Nice!
021cf66
to
beaf5b1
Compare
Array::get_max_buffer_sizes
.
beaf5b1
to
0658a54
Compare
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.
Just removed an old unit test, will merge after TileDB-Inc/TileDB-Go#349 👍
This PR removes all the code associated with
Array::get_max_buffer_sizes
.This means Array bookkeeping members, functions for calculating maximum buffer sizes based on a subarray, capnp struct, rest client functions.
The
tiledb_serialize_array_max_buffer_sizes
C API was kept until downstreams migrate, but its implementation was updated to always fail.Soon will follow a PR in the REST server for removing the handling code of this request.
TYPE: NO_HISTORY