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

VFS: Support GCS #1585

Merged
merged 1 commit into from
Apr 9, 2020
Merged

VFS: Support GCS #1585

merged 1 commit into from
Apr 9, 2020

Conversation

joe-maley
Copy link
Contributor

This is the final functional patch planned for initial GCS support. The GCS
backend is already complete, this change just enables the VFS to use it when
the core is built with GCS.

@joe-maley joe-maley requested a review from ihnorton April 8, 2020 18:54
@joe-maley joe-maley force-pushed the jpm/gcs-vfs branch 2 times, most recently from 96407b5 to a280de2 Compare April 9, 2020 12:29
tiledb/sm/filesystem/vfs.cc Show resolved Hide resolved
@ihnorton
Copy link
Member

ihnorton commented Apr 9, 2020

Comment about config, otherwise LGTM once the tests pass (some tests segfaulted due to context creation failure)

@joe-maley joe-maley force-pushed the jpm/gcs-vfs branch 2 times, most recently from 4b99f1d to 0c54aed Compare April 9, 2020 16:16
scripts/azure-linux_mac.yml Outdated Show resolved Hide resolved
@joe-maley joe-maley force-pushed the jpm/gcs-vfs branch 2 times, most recently from 3bb1d61 to 73a6959 Compare April 9, 2020 18:15
scripts/azure-linux_mac.yml Show resolved Hide resolved
scripts/azure-linux_mac.yml Outdated Show resolved Hide resolved
This is the final functional patch planned for initial GCS support. The GCS
backend is already complete, this change just enables the VFS to use it when
the core is built with GCS.
@joe-maley joe-maley merged commit 1e756ef into dev Apr 9, 2020
@joe-maley joe-maley deleted the jpm/gcs-vfs branch April 9, 2020 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants