-
Notifications
You must be signed in to change notification settings - Fork 4.2k
GH-32123: [R] Expose azure blob filesystem #49553
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
Merged
jonkeane
merged 89 commits into
apache:main
from
marberts:32123-expose-azure-blob-filesystem
Jul 13, 2026
Merged
Changes from all commits
Commits
Show all changes
89 commits
Select commit
Hold shift + click to select a range
76db447
Add AzureBlobFileSystem placeholder, verify devtools::document() beha…
Collinbrown95 943cbfe
add simple test function to work through codegen.R
Collinbrown95 5a40fde
temporarily "force" build DARROW_R_WITH_AZUREFS build flag.
Collinbrown95 4681b41
Added c++ stub
marberts 5769649
Updated codegen
marberts 2dbf5c7
Added a comment
marberts 936ee78
cleanup azurefs test function code
Collinbrown95 778c606
document instructions to start local azurite container
Collinbrown95 83ea378
add arrow_with_azure helper following convention for s3/gcp
Collinbrown95 33de88c
add ARROW_AZURE flag to nixlibs.R
Collinbrown95 38e4492
debug first argument check
Collinbrown95 8ba0eaa
Renamed R6 class correctly
marberts cbbd654
Added endpoint + key, token, and default authentication
marberts 878a450
Finished logical for AzureFileSystem to match pyarrow
marberts 9134cb7
standardize on ARROW_R_WITH_AZURE instead of ARROW_R_WITH_AZUREFS
Collinbrown95 0ac57a8
standardize on ARROW_R_WITH_AZURE
Collinbrown95 e8fcd43
Turn on ARROW_AZURE flag in nixlibs.R
Collinbrown95 8ffc149
drop temporary arrow env var hack
Collinbrown95 517750d
temporary documentation of what I've tried so far
Collinbrown95 8bf5b05
Add TODO note in configure script to remove hard-coded link flags
Collinbrown95 cc79940
initial filesystem tests
Collinbrown95 b5d0703
uncomment line 256 of filesystem.cpp
Collinbrown95 96e3744
checkpoint: resolved segfault error
Collinbrown95 d365ac3
skip test_filesystem tests that rely on being able to connect directl…
Collinbrown95 f274b46
Add most test cases from test_filesystem and recreate a couple that w…
Collinbrown95 4e70734
rename az_bucket to az_container
Collinbrown95 89e9adc
check that azurite is installed as precondition for test-azure.R script.
Collinbrown95 9bf81a5
add setup code to start azurite from the test-azure.R script, then ki…
Collinbrown95 835879b
run air formatter
Collinbrown95 cf1a3b5
add documentation to az_container.
Collinbrown95 321de88
docs: Updated documentation for AzureFileSystem and updated vignette …
marberts bab125a
Updated installation vignettes to include Azure
marberts e93645f
Updated install scripts
marberts 6f70268
add tests for valid and invalid combinations of options to AzureFileS…
Collinbrown95 1c886cd
Ran pre-commit hooks
marberts 0e87a5d
Removed tmp.md
marberts 143a129
wrap credential configuration methods with StopIfNotOk
Collinbrown95 dbb33a4
move link flags to arrow_built_with ARROW_AZURE block in configure sc…
Collinbrown95 9137668
fix error message to check in test for empty call to AzureFileSystem$…
Collinbrown95 4501816
Updated docs to include libxml2
marberts e2654af
add simple test function to work through codegen.R
Collinbrown95 6e20676
cleanup azurefs test function code
Collinbrown95 d09817b
Updated filesystem.cpp
marberts fa6cc9e
Updated build scripts
marberts cf1050a
Update configure.win: fix typo
Collinbrown95 e71257c
Updated CI setup scripts for windows
marberts 51e01d4
Trying PKGBUILD and windows build script again
marberts e0a0a17
Updated PKGBUILD
marberts 7be0a70
debug ci: add azure dependencies that CI tries to build from source.
Collinbrown95 0c3ff58
Removed tmp.md
marberts 7d1aad9
add install script to build azure-sdk-for-cpp dependencies from source.
8d1c841
add simple test function to work through codegen.R
Collinbrown95 d3ce08d
temporarily "force" build DARROW_R_WITH_AZUREFS build flag.
Collinbrown95 68c01e0
Added c++ stub
marberts 7d29231
cleanup azurefs test function code
Collinbrown95 3892e4c
document instructions to start local azurite container
Collinbrown95 f7c049f
debug first argument check
Collinbrown95 bc9e0d9
Added endpoint + key, token, and default authentication
marberts 9931e26
Finished logical for AzureFileSystem to match pyarrow
marberts a39c227
standardize on ARROW_R_WITH_AZURE
Collinbrown95 8f3df68
temporary documentation of what I've tried so far
Collinbrown95 1106490
Add TODO note in configure script to remove hard-coded link flags
Collinbrown95 e784d61
uncomment line 256 of filesystem.cpp
Collinbrown95 17fc26e
checkpoint: resolved segfault error
Collinbrown95 cb86452
Add most test cases from test_filesystem and recreate a couple that w…
Collinbrown95 69065d6
check that azurite is installed as precondition for test-azure.R script.
Collinbrown95 a6301e1
Ran pre-commit hooks
marberts 1d50661
Removed tmp.md
marberts 263b462
wrap credential configuration methods with StopIfNotOk
Collinbrown95 cb19431
move link flags to arrow_built_with ARROW_AZURE block in configure sc…
Collinbrown95 3486281
add simple test function to work through codegen.R
Collinbrown95 2bc10ba
cleanup azurefs test function code
Collinbrown95 169ca19
Trying PKGBUILD and windows build script again
marberts d39eef4
Updated PKGBUILD
marberts fb850f1
debug ci: add azure dependencies that CI tries to build from source.
Collinbrown95 7619f01
Removed tmp.md
marberts ff68542
Removed windows build and added macos
marberts 9ea60df
azure off for windows ci
marberts 5a3eed4
Disable azure in configure.win
marberts 8f5d9ba
Add placeholder comment to flag present issues with MinGW and the Azu…
Collinbrown95 addcfa5
remove debugging script to install azure sdk from source
Collinbrown95 b58fcc5
Updated vignettes to reflect Azure off on Windows
marberts d5cd943
Update github.packages.yml
marberts 211c1fc
Fixing lint in filesystem.cpp
marberts c7dd3e5
Fixed warning in cpp code
marberts 24c918b
Added build info for azure to tasks for mac + a couple code comments
marberts a8ad6a9
Fixed line endings on github.packages yaml
marberts 84f3b74
libxml2 warning in with_cloud_support() is now conditional on azure b…
marberts 9dd6793
Add libxml2
jonkeane 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 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
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
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 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
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
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.
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.
These are holdovers from when you were trying these on Windows, yeah?
Uh oh!
There was an error while loading. Please reload this page.
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.
Yes, I left these commented out in case progress gets made in the future around the incompatibility with mingw and the azure SDK stuff. Happy to either remove them or make it clearer in the comments.