jonkeane commented on a change in pull request #11680:
URL: https://github.com/apache/arrow/pull/11680#discussion_r747695123
##########
File path: r/src/filesystem.cpp
##########
@@ -287,6 +287,9 @@ std::shared_ptr<fs::S3FileSystem> fs___S3FileSystem__create(
std::string session_name = "", std::string external_id = "", int
load_frequency = 900,
std::string region = "", std::string endpoint_override = "", std::string
scheme = "",
bool background_writes = true) {
+ // We need to ensure that S3 is initialized before we start messing with the
+ // options
+ StopIfNotOk(fs::EnsureS3Initialized());
Review comment:
We might not need this to be _this_ high — I believe the segfault was
`s3_opts = fs::S3Options::Defaults();` below, but it shouldn't hurt to have it
up here
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
