Skip to content

Commit

Permalink
apacheGH-40562: [C++] Repair FileSystem merge error
Browse files Browse the repository at this point in the history
  • Loading branch information
bkietz committed Mar 14, 2024
1 parent b235f83 commit e6c55fd
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 7 deletions.
8 changes: 4 additions & 4 deletions cpp/src/arrow/filesystem/azurefs.cc
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ AzureOptions::AzureOptions() = default;

AzureOptions::~AzureOptions() = default;

void AzureOptions::ExtractFromUriSchemeAndHierPart(const arrow::internal::Uri& uri,
void AzureOptions::ExtractFromUriSchemeAndHierPart(const Uri& uri,
std::string* out_path) {
const auto host = uri.host();
std::string path;
Expand Down Expand Up @@ -99,7 +99,7 @@ void AzureOptions::ExtractFromUriSchemeAndHierPart(const arrow::internal::Uri& u
}
}

Status AzureOptions::ExtractFromUriQuery(const arrow::internal::Uri& uri) {
Status AzureOptions::ExtractFromUriQuery(const Uri& uri) {
const auto account_key = uri.password();
std::optional<CredentialKind> credential_kind;
std::optional<std::string> credential_kind_value;
Expand Down Expand Up @@ -216,7 +216,7 @@ Status AzureOptions::ExtractFromUriQuery(const arrow::internal::Uri& uri) {
return Status::OK();
}

Result<AzureOptions> AzureOptions::FromUri(const arrow::internal::Uri& uri,
Result<AzureOptions> AzureOptions::FromUri(const Uri& uri,
std::string* out_path) {
AzureOptions options;
options.ExtractFromUriSchemeAndHierPart(uri, out_path);
Expand All @@ -226,7 +226,7 @@ Result<AzureOptions> AzureOptions::FromUri(const arrow::internal::Uri& uri,

Result<AzureOptions> AzureOptions::FromUri(const std::string& uri_string,
std::string* out_path) {
arrow::internal::Uri uri;
Uri uri;
RETURN_NOT_OK(uri.Parse(uri_string));
return FromUri(uri, out_path);
}
Expand Down
6 changes: 3 additions & 3 deletions cpp/src/arrow/filesystem/azurefs.h
Original file line number Diff line number Diff line change
Expand Up @@ -127,9 +127,9 @@ struct ARROW_EXPORT AzureOptions {
~AzureOptions();

private:
void ExtractFromUriSchemeAndHierPart(const arrow::internal::Uri& uri,
void ExtractFromUriSchemeAndHierPart(const Uri& uri,
std::string* out_path);
Status ExtractFromUriQuery(const arrow::internal::Uri& uri);
Status ExtractFromUriQuery(const Uri& uri);

public:
/// \brief Construct a new AzureOptions from an URI.
Expand Down Expand Up @@ -175,7 +175,7 @@ struct ARROW_EXPORT AzureOptions {
/// AzureOptions::ConfigureClientSecretCredential() is called.
/// * client_secret: You must specify "tenant_id" and "client_id"
/// too. AzureOptions::ConfigureClientSecretCredential() is called.
static Result<AzureOptions> FromUri(const arrow::internal::Uri& uri,
static Result<AzureOptions> FromUri(const Uri& uri,
std::string* out_path);
static Result<AzureOptions> FromUri(const std::string& uri, std::string* out_path);

Expand Down

0 comments on commit e6c55fd

Please sign in to comment.