Skip to content

Commit eff7ef0

Browse files
committed
Take into account possible external s3-compatible service configured
Fix comment
1 parent 4faccf6 commit eff7ef0

1 file changed

Lines changed: 7 additions & 2 deletions

File tree

cpp/src/arrow/filesystem/s3_test_util.cc

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -194,8 +194,13 @@ void MinioTestEnvironment::SetUp() {
194194

195195
bool MinioTestEnvironment::IsAvailable() {
196196
if (!checked_) {
197-
::arrow::util::Process process;
198-
available_ = process.SetExecutable(kMinioExecutableName).ok();
197+
// If external S3-compatible service is configured there's no need to check for Minio
198+
available_ = std::getenv(kEnvConnectString) && std::getenv(kEnvAccessKey) &&
199+
std::getenv(kEnvSecretKey);
200+
if (!available_) {
201+
::arrow::util::Process process;
202+
available_ = process.SetExecutable(kMinioExecutableName).ok();
203+
}
199204
checked_ = true;
200205
}
201206
return available_;

0 commit comments

Comments
 (0)