-
Notifications
You must be signed in to change notification settings - Fork 202
Update to use post_array_schema_from_rest.
#5181
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
Draft
shaunrd0
wants to merge
36
commits into
main
Choose a base branch
from
pd/sc-32991/use-handle-load-array-request
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from 32 commits
Commits
Show all changes
36 commits
Select commit
Hold shift + click to select a range
34571cb
Implement tiledb_handle_load_array_schema_request
davisp 2fbef25
Switch to `post_array_schema_from_rest`
davisp 38883d2
Add tiledb_array_schema_get_enumeration
davisp a7c849d
[WIP] Informal feed back on the load schema API
davisp 8eda237
Merge remote-tracking branch 'origin/dev' into pd/sc-32991/add-handle…
shaunrd0 c96ca4b
Changes after merge.
shaunrd0 bb69fdc
Regenerate capnp sources.
shaunrd0 b4cee56
Add test from @jp-dark.
shaunrd0 63e22d4
Serialize all array schemas.
shaunrd0 8cccf09
Regenerate capnp.
shaunrd0 3aa1f8d
Use post_array_schema_from_rest.
shaunrd0 5b1f8ec
Update to use schema_load_with_options.
shaunrd0 61a1d39
Update tests to use VFSTestSetup.
shaunrd0 95abe63
Merge remote-tracking branch 'origin/dev' into pd/sc-32991/use-handle…
shaunrd0 73eadd4
Port regression test to REST CI.
shaunrd0 2783cbc
Test with REST CI.
shaunrd0 57ccb1f
Remove get_array_schema_from_rest, update docs.
shaunrd0 a3b5c44
Fix redefinition for stub function.
shaunrd0 9b1d3ae
Remove bool from LoadArraySchemaRequest.
shaunrd0 10f789b
Regenerate capnp.
shaunrd0 a0bf2d2
Fix config option name.
shaunrd0 5072f3c
Merge remote-tracking branch 'origin/dev' into pd/sc-32991/use-handle…
shaunrd0 e1020fc
Changes after merge.
shaunrd0 f322b78
Changes from review.
shaunrd0 afa9175
Move logic out of tiledb.cc.
shaunrd0 c424e77
Check all array schemas.
shaunrd0 e9d8191
Add docs and rename function.
shaunrd0 ddd3ac9
Merge remote-tracking branch 'origin/dev' into pd/sc-32991/use-handle…
shaunrd0 b00936b
Fix REST CI segfault.
shaunrd0 aa494ce
Fix CI.
shaunrd0 e05e701
Merge remote-tracking branch 'origin/dev' into pd/sc-32991/use-handle…
shaunrd0 2cbf009
Merge branch 'dev' into pd/sc-32991/use-handle-load-array-request
ypatia 4cde5b6
Remove test branch from CI.
shaunrd0 a5687f0
Merge remote-tracking branch 'origin/dev' into pd/sc-32991/use-handle…
shaunrd0 54b9305
Load enumerations for all array schemas.
shaunrd0 6f108ee
Merge remote-tracking branch 'origin/dev' into pd/sc-32991/use-handle…
shaunrd0 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.