You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "pitrou (via GitHub)" <gi...@apache.org> on 2024/05/23 14:09:03 UTC
[PR] GH-41797: [C++][S3] Remove GetBucketRegion hack for never AWS SDK versions [arrow]
pitrou opened a new pull request, #41798:
URL: https://github.com/apache/arrow/pull/41798
<!--
Thanks for opening a pull request!
If this is your first pull request you can find detailed information on how
to contribute here:
* [New Contributor's Guide](https://arrow.apache.org/docs/dev/developers/guide/step_by_step/pr_lifecycle.html#reviews-and-merge-of-the-pull-request)
* [Contributing Overview](https://arrow.apache.org/docs/dev/developers/overview.html)
If this is not a [minor PR](https://github.com/apache/arrow/blob/main/CONTRIBUTING.md#Minor-Fixes). Could you open an issue for this pull request on GitHub? https://github.com/apache/arrow/issues/new/choose
Opening GitHub issues ahead of time contributes to the [Openness](http://theapacheway.com/open/#:~:text=Openness%20allows%20new%20users%20the,must%20happen%20in%20the%20open.) of the Apache Arrow project.
Then could you also rename the pull request title in the following format?
GH-${GITHUB_ISSUE_ID}: [${COMPONENT}] ${SUMMARY}
or
MINOR: [${COMPONENT}] ${SUMMARY}
In the case of PARQUET issues on JIRA the title also supports:
PARQUET-${JIRA_ISSUE_ID}: [${COMPONENT}] ${SUMMARY}
-->
### Rationale for this change
To get the region a S3 bucket resides on, it is required to issue a HeadBucket region and parse the response headers for a certain header value.
Unfortunately, the AWS SDK doesn't let us access arbitrary headers on successful responses for S3 model requests, which had us implement a workaround by calling lower-level SDK APIs.
However, the SDK recently added a `GetBucketRegion` method on `HeadBucketRequest`, which obsoletes the need for this workaround. We now use this method if the available AWS SDK version is recent enough.
### Are these changes tested?
By existing tests on the various CI configurations.
### Are there any user-facing changes?
No.
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] GH-41797: [C++][S3] Remove GetBucketRegion hack for never AWS SDK versions [arrow]
Posted by "pitrou (via GitHub)" <gi...@apache.org>.
pitrou commented on PR #41798:
URL: https://github.com/apache/arrow/pull/41798#issuecomment-2127226459
@github-actions crossbow submit -g cpp
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] GH-41797: [C++][S3] Remove GetBucketRegion hack for newer AWS SDK versions [arrow]
Posted by "pitrou (via GitHub)" <gi...@apache.org>.
pitrou commented on PR #41798:
URL: https://github.com/apache/arrow/pull/41798#issuecomment-2150419286
I'll merge this as the CI failures are unrelated.
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] GH-41797: [C++][S3] Remove GetBucketRegion hack for never AWS SDK versions [arrow]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #41798:
URL: https://github.com/apache/arrow/pull/41798#issuecomment-2127232259
Revision: e4e6ef221976489f7b8379fc5c9fbdab3de6029f
Submitted crossbow builds: [ursacomputing/crossbow @ actions-1f62e4c0dc](https://github.com/ursacomputing/crossbow/branches/all?query=actions-1f62e4c0dc)
|Task|Status|
|----|------|
|test-alpine-linux-cpp|[](https://github.com/ursacomputing/crossbow/actions/runs/9209570665/job/25334484016)|
|test-build-cpp-fuzz|[](https://github.com/ursacomputing/crossbow/actions/runs/9209570638/job/25334483633)|
|test-conda-cpp|[](https://github.com/ursacomputing/crossbow/actions/runs/9209571097/job/25334485247)|
|test-conda-cpp-valgrind|[](https://github.com/ursacomputing/crossbow/actions/runs/9209570616/job/25334483585)|
|test-cuda-cpp|[](https://github.com/ursacomputing/crossbow/actions/runs/9209570317/job/25334482377)|
|test-debian-12-cpp-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209570387/job/25334482747)|
|test-debian-12-cpp-i386|[](https://github.com/ursacomputing/crossbow/actions/runs/9209570309/job/25334482058)|
|test-fedora-39-cpp|[](https://github.com/ursacomputing/crossbow/actions/runs/9209570402/job/25334482990)|
|test-ubuntu-20.04-cpp|[](https://github.com/ursacomputing/crossbow/actions/runs/9209570892/job/25334484663)|
|test-ubuntu-20.04-cpp-bundled|[](https://github.com/ursacomputing/crossbow/actions/runs/9209570373/job/25334482758)|
|test-ubuntu-20.04-cpp-minimal-with-formats|[](https://github.com/ursacomputing/crossbow/actions/runs/9209570216/job/25334481957)|
|test-ubuntu-20.04-cpp-thread-sanitizer|[](https://github.com/ursacomputing/crossbow/actions/runs/9209570082/job/25334481582)|
|test-ubuntu-22.04-cpp|[](https://github.com/ursacomputing/crossbow/actions/runs/9209570390/job/25334483002)|
|test-ubuntu-22.04-cpp-20|[](https://github.com/ursacomputing/crossbow/actions/runs/9209570722/job/25334484460)|
|test-ubuntu-22.04-cpp-emscripten|[](https://github.com/ursacomputing/crossbow/actions/runs/9209570129/job/25334481584)|
|test-ubuntu-22.04-cpp-no-threading|[](https://github.com/ursacomputing/crossbow/actions/runs/9209570680/job/25334483878)|
|test-ubuntu-24.04-cpp|[](https://github.com/ursacomputing/crossbow/actions/runs/9209570194/job/25334482097)|
|test-ubuntu-24.04-cpp-gcc-14|[](https://github.com/ursacomputing/crossbow/actions/runs/9209570487/job/25334483452)|
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] GH-41797: [C++][S3] Remove GetBucketRegion hack for newer AWS SDK versions [arrow]
Posted by "conbench-apache-arrow[bot] (via GitHub)" <gi...@apache.org>.
conbench-apache-arrow[bot] commented on PR #41798:
URL: https://github.com/apache/arrow/pull/41798#issuecomment-2160838929
After merging your PR, Conbench analyzed the 6 benchmarking runs that have been run so far on merge-commit 51bc2a61c90a89a29dacacbada190aa06f232271.
There were no benchmark performance regressions. 🎉
The [full Conbench report](https://github.com/apache/arrow/runs/26079878522) has more details. It also includes information about 7 possible false positives for unstable benchmarks that are known to sometimes produce them.
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] GH-41797: [C++][S3] Remove GetBucketRegion hack for newer AWS SDK versions [arrow]
Posted by "bkietz (via GitHub)" <gi...@apache.org>.
bkietz commented on code in PR #41798:
URL: https://github.com/apache/arrow/pull/41798#discussion_r1613704875
##########
cpp/src/arrow/filesystem/s3fs.cc:
##########
@@ -601,44 +601,81 @@ class S3Client : public Aws::S3::S3Client {
public:
using Aws::S3::S3Client::S3Client;
+ static inline constexpr auto kBucketRegionHeaderName = "x-amz-bucket-region";
+
+ std::string GetBucketRegionFromHeaders(
+ const Aws::Http::HeaderValueCollection& headers) {
+ const auto it = headers.find(ToAwsString(kBucketRegionHeaderName));
+ if (it != headers.end()) {
+ return std::string(FromAwsString(it->second));
+ }
+ return std::string();
Review Comment:
It seems slightly suspicious that we're not returning an error in this case
```suggestion
return Status::IOError("When resolving region for bucket '", request.GetBucket(),
"': missing 'x-amz-bucket-region' header in response");
```
I think this would be less surprising, and below where we need an IOError we could write
```c++
template <typename ErrorType>
Result<std::string> GetBucketRegionFromError(
const std::string& bucket, const Aws::Client::AWSError<ErrorType>& error) {
Result<std::string> region = GetBucketRegionFromHeaders(error.GetResponseHeaders());
if (!region.ok() && error.GetResponseCode() == Aws::Http::HttpResponseCode::NOT_FOUND) {
return Status::IOError("Bucket '", bucket, "' not found");
}
return region;
}
```
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] GH-41797: [C++][S3] Remove GetBucketRegion hack for newer AWS SDK versions [arrow]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #41798:
URL: https://github.com/apache/arrow/pull/41798#issuecomment-2150245487
Revision: 0e7e0f1efec1f8172c02239597eae3217919c30b
Submitted crossbow builds: [ursacomputing/crossbow @ actions-30c784565b](https://github.com/ursacomputing/crossbow/branches/all?query=actions-30c784565b)
|Task|Status|
|----|------|
|example-python-minimal-build-fedora-conda|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229528/job/25846287791)|
|example-python-minimal-build-ubuntu-venv|[](https://github.com/ursacomputing/crossbow/actions/runs/9386230356/job/25846342083)|
|test-conda-python-3.10|[](https://github.com/ursacomputing/crossbow/actions/runs/9386230209/job/25846341024)|
|test-conda-python-3.10-cython2|[](https://github.com/ursacomputing/crossbow/actions/runs/9386230329/job/25846342097)|
|test-conda-python-3.10-hdfs-2.9.2|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229148/job/25846285151)|
|test-conda-python-3.10-hdfs-3.2.1|[](https://github.com/ursacomputing/crossbow/actions/runs/9386230036/job/25846291777)|
|test-conda-python-3.10-pandas-latest|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229426/job/25846287503)|
|test-conda-python-3.10-pandas-nightly|[](https://github.com/ursacomputing/crossbow/actions/runs/9386230496/job/25846342962)|
|test-conda-python-3.10-spark-v3.5.0|[](https://github.com/ursacomputing/crossbow/actions/runs/9386230457/job/25846342907)|
|test-conda-python-3.10-substrait|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229288/job/25846285769)|
|test-conda-python-3.11|[](https://github.com/ursacomputing/crossbow/actions/runs/9386230461/job/25846342930)|
|test-conda-python-3.11-dask-latest|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229935/job/25846293203)|
|test-conda-python-3.11-dask-upstream_devel|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229234/job/25846286171)|
|test-conda-python-3.11-hypothesis|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229636/job/25846288944)|
|test-conda-python-3.11-pandas-upstream_devel|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229592/job/25846288122)|
|test-conda-python-3.11-spark-master|[](https://github.com/ursacomputing/crossbow/actions/runs/9386230433/job/25846342330)|
|test-conda-python-3.12|[](https://github.com/ursacomputing/crossbow/actions/runs/9386230377/job/25846342080)|
|test-conda-python-3.8|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229436/job/25846287508)|
|test-conda-python-3.8-pandas-1.0|[](https://github.com/ursacomputing/crossbow/actions/runs/9386230173/job/25846291861)|
|test-conda-python-3.8-spark-v3.5.0|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229637/job/25846289019)|
|test-conda-python-3.9|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229673/job/25846290235)|
|test-conda-python-3.9-pandas-latest|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229534/job/25846287826)|
|test-cuda-python|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229901/job/25846290546)|
|test-debian-12-python-3-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229894/job/25846290521)|
|test-debian-12-python-3-i386|[](https://github.com/ursacomputing/crossbow/actions/runs/9386230755/job/25846343740)|
|test-fedora-39-python-3|[](https://github.com/ursacomputing/crossbow/actions/runs/9386230233/job/25846341307)|
|test-ubuntu-20.04-python-3|[](https://github.com/ursacomputing/crossbow/actions/runs/9386230581/job/25846343786)|
|test-ubuntu-22.04-python-3|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229462/job/25846287546)|
|wheel-macos-big-sur-cp310-arm64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229595/job/25846288045)|
|wheel-macos-big-sur-cp311-arm64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386230247/job/25846341448)|
|wheel-macos-big-sur-cp312-arm64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229099/job/25846284603)|
|wheel-macos-big-sur-cp38-arm64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386230554/job/25846343780)|
|wheel-macos-big-sur-cp39-arm64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229277/job/25846286478)|
|wheel-macos-catalina-cp310-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229936/job/25846291489)|
|wheel-macos-catalina-cp311-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386230383/job/25846342362)|
|wheel-macos-catalina-cp312-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386230073/job/25846293358)|
|wheel-macos-catalina-cp38-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386230518/job/25846343183)|
|wheel-macos-catalina-cp39-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229471/job/25846286533)|
|wheel-manylinux-2-28-cp310-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229452/job/25846286532)|
|wheel-manylinux-2-28-cp310-arm64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229564/job/25846287868)|
|wheel-manylinux-2-28-cp311-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386230290/job/25846341473)|
|wheel-manylinux-2-28-cp311-arm64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229970/job/25846291556)|
|wheel-manylinux-2-28-cp312-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229779/job/25846290239)|
|wheel-manylinux-2-28-cp312-arm64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229993/job/25846291654)|
|wheel-manylinux-2-28-cp38-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229317/job/25846286511)|
|wheel-manylinux-2-28-cp38-arm64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229974/job/25846291559)|
|wheel-manylinux-2-28-cp39-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386230079/job/25846291872)|
|wheel-manylinux-2-28-cp39-arm64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386230217/job/25846341115)|
|wheel-manylinux-2014-cp310-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229792/job/25846291254)|
|wheel-manylinux-2014-cp310-arm64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229801/job/25846291270)|
|wheel-manylinux-2014-cp311-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229052/job/25846284550)|
|wheel-manylinux-2014-cp311-arm64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229796/job/25846290255)|
|wheel-manylinux-2014-cp312-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386230349/job/25846342064)|
|wheel-manylinux-2014-cp312-arm64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229918/job/25846291352)|
|wheel-manylinux-2014-cp38-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229316/job/25846285596)|
|wheel-manylinux-2014-cp38-arm64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386230302/job/25846341616)|
|wheel-manylinux-2014-cp39-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386230509/job/25846342966)|
|wheel-manylinux-2014-cp39-arm64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229360/job/25846287571)|
|wheel-windows-cp310-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386230517/job/25846343239)|
|wheel-windows-cp311-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229218/job/25846286175)|
|wheel-windows-cp312-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229578/job/25846287940)|
|wheel-windows-cp38-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229851/job/25846290432)|
|wheel-windows-cp39-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229060/job/25846284559)|
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] GH-41797: [C++][S3] Remove GetBucketRegion hack for newer AWS SDK versions [arrow]
Posted by "pitrou (via GitHub)" <gi...@apache.org>.
pitrou commented on code in PR #41798:
URL: https://github.com/apache/arrow/pull/41798#discussion_r1617548412
##########
cpp/src/arrow/filesystem/s3fs.cc:
##########
@@ -601,44 +601,81 @@ class S3Client : public Aws::S3::S3Client {
public:
using Aws::S3::S3Client::S3Client;
+ static inline constexpr auto kBucketRegionHeaderName = "x-amz-bucket-region";
+
+ std::string GetBucketRegionFromHeaders(
+ const Aws::Http::HeaderValueCollection& headers) {
+ const auto it = headers.find(ToAwsString(kBucketRegionHeaderName));
+ if (it != headers.end()) {
+ return std::string(FromAwsString(it->second));
+ }
+ return std::string();
Review Comment:
That's because we return two different error messages depending on the code path, see the `GetBucketRegionFromHeaders` call sites.
1. When the headers come from a successful response, the error message is the generic one you propose
2. When the headers come from an error response, we reuse the original error message.
Does that sound ok?
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] GH-41797: [C++][S3] Remove GetBucketRegion hack for never AWS SDK versions [arrow]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #41798:
URL: https://github.com/apache/arrow/pull/41798#issuecomment-2127224577
:warning: GitHub issue #41797 **has been automatically assigned in GitHub** to PR creator.
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] GH-41797: [C++][S3] Remove GetBucketRegion hack for newer AWS SDK versions [arrow]
Posted by "pitrou (via GitHub)" <gi...@apache.org>.
pitrou commented on PR #41798:
URL: https://github.com/apache/arrow/pull/41798#issuecomment-2150229134
@github-actions crossbow submit -g python -g wheel
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] GH-41797: [C++][S3] Remove GetBucketRegion hack for newer AWS SDK versions [arrow]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #41798:
URL: https://github.com/apache/arrow/pull/41798#issuecomment-2150239165
Revision: 0e7e0f1efec1f8172c02239597eae3217919c30b
Submitted crossbow builds: [ursacomputing/crossbow @ actions-8494188154](https://github.com/ursacomputing/crossbow/branches/all?query=actions-8494188154)
|Task|Status|
|----|------|
|test-alpine-linux-cpp|[](https://github.com/ursacomputing/crossbow/actions/runs/9386228698/job/25846282459)|
|test-build-cpp-fuzz|[](https://github.com/ursacomputing/crossbow/actions/runs/9386228836/job/25846283324)|
|test-conda-cpp|[](https://github.com/ursacomputing/crossbow/actions/runs/9386228929/job/25846284025)|
|test-conda-cpp-valgrind|[](https://github.com/ursacomputing/crossbow/actions/runs/9386228466/job/25846281388)|
|test-cuda-cpp|[](https://github.com/ursacomputing/crossbow/actions/runs/9386229049/job/25846284592)|
|test-debian-12-cpp-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9386228709/job/25846282615)|
|test-debian-12-cpp-i386|[](https://github.com/ursacomputing/crossbow/actions/runs/9386228652/job/25846282185)|
|test-fedora-39-cpp|[](https://github.com/ursacomputing/crossbow/actions/runs/9386228660/job/25846282184)|
|test-ubuntu-20.04-cpp|[](https://github.com/ursacomputing/crossbow/actions/runs/9386228849/job/25846283351)|
|test-ubuntu-20.04-cpp-bundled|[](https://github.com/ursacomputing/crossbow/actions/runs/9386228481/job/25846281353)|
|test-ubuntu-20.04-cpp-minimal-with-formats|[](https://github.com/ursacomputing/crossbow/actions/runs/9386228477/job/25846281389)|
|test-ubuntu-20.04-cpp-thread-sanitizer|[](https://github.com/ursacomputing/crossbow/actions/runs/9386228864/job/25846283985)|
|test-ubuntu-22.04-cpp|[](https://github.com/ursacomputing/crossbow/actions/runs/9386228966/job/25846283984)|
|test-ubuntu-22.04-cpp-20|[](https://github.com/ursacomputing/crossbow/actions/runs/9386228853/job/25846283558)|
|test-ubuntu-22.04-cpp-emscripten|[](https://github.com/ursacomputing/crossbow/actions/runs/9386228507/job/25846281492)|
|test-ubuntu-22.04-cpp-no-threading|[](https://github.com/ursacomputing/crossbow/actions/runs/9386228682/job/25846282281)|
|test-ubuntu-24.04-cpp|[](https://github.com/ursacomputing/crossbow/actions/runs/9386228808/job/25846283195)|
|test-ubuntu-24.04-cpp-gcc-14|[](https://github.com/ursacomputing/crossbow/actions/runs/9386228933/job/25846283991)|
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] GH-41797: [C++][S3] Remove GetBucketRegion hack for never AWS SDK versions [arrow]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #41798:
URL: https://github.com/apache/arrow/pull/41798#issuecomment-2127255144
Revision: e4e6ef221976489f7b8379fc5c9fbdab3de6029f
Submitted crossbow builds: [ursacomputing/crossbow @ actions-b9eda013cf](https://github.com/ursacomputing/crossbow/branches/all?query=actions-b9eda013cf)
|Task|Status|
|----|------|
|example-python-minimal-build-fedora-conda|[](https://github.com/ursacomputing/crossbow/actions/runs/9209720871/job/25334984338)|
|example-python-minimal-build-ubuntu-venv|[](https://github.com/ursacomputing/crossbow/actions/runs/9209721792/job/25334988596)|
|test-conda-python-3.10|[](https://github.com/ursacomputing/crossbow/actions/runs/9209720758/job/25334983244)|
|test-conda-python-3.10-cython2|[](https://github.com/ursacomputing/crossbow/actions/runs/9209721459/job/25334987774)|
|test-conda-python-3.10-hdfs-2.9.2|[](https://github.com/ursacomputing/crossbow/actions/runs/9209720759/job/25334983700)|
|test-conda-python-3.10-hdfs-3.2.1|[](https://github.com/ursacomputing/crossbow/actions/runs/9209722444/job/25334992293)|
|test-conda-python-3.10-pandas-latest|[](https://github.com/ursacomputing/crossbow/actions/runs/9209720331/job/25334982011)|
|test-conda-python-3.10-pandas-nightly|[](https://github.com/ursacomputing/crossbow/actions/runs/9209721830/job/25334989731)|
|test-conda-python-3.10-spark-v3.5.0|[](https://github.com/ursacomputing/crossbow/actions/runs/9209721022/job/25334984871)|
|test-conda-python-3.10-substrait|[](https://github.com/ursacomputing/crossbow/actions/runs/9209720322/job/25334982031)|
|test-conda-python-3.11|[](https://github.com/ursacomputing/crossbow/actions/runs/9209722697/job/25334993794)|
|test-conda-python-3.11-dask-latest|[](https://github.com/ursacomputing/crossbow/actions/runs/9209721051/job/25334985228)|
|test-conda-python-3.11-dask-upstream_devel|[](https://github.com/ursacomputing/crossbow/actions/runs/9209721726/job/25334988392)|
|test-conda-python-3.11-hypothesis|[](https://github.com/ursacomputing/crossbow/actions/runs/9209720461/job/25334982441)|
|test-conda-python-3.11-pandas-upstream_devel|[](https://github.com/ursacomputing/crossbow/actions/runs/9209721241/job/25334985752)|
|test-conda-python-3.11-spark-master|[](https://github.com/ursacomputing/crossbow/actions/runs/9209721435/job/25334986781)|
|test-conda-python-3.12|[](https://github.com/ursacomputing/crossbow/actions/runs/9209721532/job/25334986810)|
|test-conda-python-3.8|[](https://github.com/ursacomputing/crossbow/actions/runs/9209722134/job/25334989965)|
|test-conda-python-3.8-pandas-1.0|[](https://github.com/ursacomputing/crossbow/actions/runs/9209722240/job/25334991134)|
|test-conda-python-3.8-spark-v3.5.0|[](https://github.com/ursacomputing/crossbow/actions/runs/9209721696/job/25334988308)|
|test-conda-python-3.9|[](https://github.com/ursacomputing/crossbow/actions/runs/9209721599/job/25334988002)|
|test-conda-python-3.9-pandas-latest|[](https://github.com/ursacomputing/crossbow/actions/runs/9209721735/job/25334988326)|
|test-cuda-python|[](https://github.com/ursacomputing/crossbow/actions/runs/9209722872/job/25334994064)|
|test-debian-12-python-3-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209720692/job/25334983265)|
|test-debian-12-python-3-i386|[](https://github.com/ursacomputing/crossbow/actions/runs/9209721469/job/25334987334)|
|test-fedora-39-python-3|[](https://github.com/ursacomputing/crossbow/actions/runs/9209722010/job/25334989725)|
|test-ubuntu-20.04-python-3|[](https://github.com/ursacomputing/crossbow/actions/runs/9209722618/job/25334993414)|
|test-ubuntu-22.04-python-3|[](https://github.com/ursacomputing/crossbow/actions/runs/9209721028/job/25334984886)|
|wheel-macos-big-sur-cp310-arm64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209722751/job/25334993792)|
|wheel-macos-big-sur-cp311-arm64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209720878/job/25334984378)|
|wheel-macos-big-sur-cp312-arm64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209722252/job/25334991151)|
|wheel-macos-big-sur-cp38-arm64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209722778/job/25334994053)|
|wheel-macos-big-sur-cp39-arm64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209721183/job/25334985745)|
|wheel-macos-catalina-cp310-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209720420/job/25334982409)|
|wheel-macos-catalina-cp311-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209722518/job/25334992377)|
|wheel-macos-catalina-cp312-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209720992/job/25334984882)|
|wheel-macos-catalina-cp38-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209720741/job/25334983473)|
|wheel-macos-catalina-cp39-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209722102/job/25334989759)|
|wheel-manylinux-2-28-cp310-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209722646/job/25334993757)|
|wheel-manylinux-2-28-cp310-arm64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209722616/job/25334993390)|
|wheel-manylinux-2-28-cp311-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209722627/job/25334993364)|
|wheel-manylinux-2-28-cp311-arm64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209721655/job/25334988100)|
|wheel-manylinux-2-28-cp312-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209723045/job/25335045877)|
|wheel-manylinux-2-28-cp312-arm64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209721043/job/25334985217)|
|wheel-manylinux-2-28-cp38-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209722398/job/25334992288)|
|wheel-manylinux-2-28-cp38-arm64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209721224/job/25334985225)|
|wheel-manylinux-2-28-cp39-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209722220/job/25334991186)|
|wheel-manylinux-2-28-cp39-arm64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209722605/job/25334993417)|
|wheel-manylinux-2014-cp310-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209721162/job/25334985735)|
|wheel-manylinux-2014-cp310-arm64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209721819/job/25334988601)|
|wheel-manylinux-2014-cp311-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209721569/job/25334988012)|
|wheel-manylinux-2014-cp311-arm64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209721683/job/25334987996)|
|wheel-manylinux-2014-cp312-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209721398/job/25334986325)|
|wheel-manylinux-2014-cp312-arm64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209720700/job/25334983220)|
|wheel-manylinux-2014-cp38-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209722495/job/25334992308)|
|wheel-manylinux-2014-cp38-arm64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209721321/job/25334986777)|
|wheel-manylinux-2014-cp39-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209721007/job/25334984876)|
|wheel-manylinux-2014-cp39-arm64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209721700/job/25334988317)|
|wheel-windows-cp310-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209722190/job/25334990163)|
|wheel-windows-cp311-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209720596/job/25334983143)|
|wheel-windows-cp312-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209720785/job/25334983520)|
|wheel-windows-cp38-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209722776/job/25334993847)|
|wheel-windows-cp39-amd64|[](https://github.com/ursacomputing/crossbow/actions/runs/9209720459/job/25334982456)|
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] GH-41797: [C++][S3] Remove GetBucketRegion hack for never AWS SDK versions [arrow]
Posted by "pitrou (via GitHub)" <gi...@apache.org>.
pitrou commented on PR #41798:
URL: https://github.com/apache/arrow/pull/41798#issuecomment-2127246970
@github-actions crossbow submit -g python -g wheel
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] GH-41797: [C++][S3] Remove GetBucketRegion hack for newer AWS SDK versions [arrow]
Posted by "pitrou (via GitHub)" <gi...@apache.org>.
pitrou commented on PR #41798:
URL: https://github.com/apache/arrow/pull/41798#issuecomment-2150229590
@github-actions crossbow submit -g cpp
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] GH-41797: [C++][S3] Remove GetBucketRegion hack for newer AWS SDK versions [arrow]
Posted by "pitrou (via GitHub)" <gi...@apache.org>.
pitrou merged PR #41798:
URL: https://github.com/apache/arrow/pull/41798
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org