Skip to content

GH-40151: [C++] Make S3 narrative test more flexible#40144

Merged
pitrou merged 2 commits intoapache:mainfrom
pitrou:minor-s3-narrative-test-improved
Feb 20, 2024
Merged

GH-40151: [C++] Make S3 narrative test more flexible#40144
pitrou merged 2 commits intoapache:mainfrom
pitrou:minor-s3-narrative-test-improved

Conversation

@pitrou
Copy link
Copy Markdown
Member

@pitrou pitrou commented Feb 19, 2024

arrow-s3fs-narrative-test allows manual testing of the S3 filesystem implementation against an actual S3 backend such as AWS.

This PR allows customizing the endpoint address, and adds a command to create a bucket for testing.

Validated with the LocalStack S3 server.

Copy link
Copy Markdown
Member

@kou kou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

This is not a MINOR change. Could you open an issue for this?

@github-actions github-actions bot added awaiting merge Awaiting merge and removed awaiting review Awaiting review labels Feb 19, 2024
@pitrou pitrou changed the title MINOR: [C++] Make S3 narrative test more flexible GH-40151: [C++] Make S3 narrative test more flexible Feb 20, 2024
`arrow-s3fs-narrative-test` allows manual testing of the S3 filesystem implementation against an actual S3 backend such as AWS.

This PR allows customizing the endpoint address, and adds a command to create a bucket for testing.

Validated with the LocalStack S3 server.
@pitrou pitrou force-pushed the minor-s3-narrative-test-improved branch from edf158e to 5368e9d Compare February 20, 2024 09:59
@pitrou pitrou merged commit 4dc3d04 into apache:main Feb 20, 2024
@pitrou pitrou removed the awaiting merge Awaiting merge label Feb 20, 2024
@pitrou pitrou deleted the minor-s3-narrative-test-improved branch February 20, 2024 10:56
@conbench-apache-arrow
Copy link
Copy Markdown

After merging your PR, Conbench analyzed the 5 benchmarking runs that have been run so far on merge-commit 4dc3d04.

There was 1 benchmark result with an error:

There were no benchmark performance regressions. 🎉

The full Conbench report has more details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[C++][S3] Improve narrative tests

3 participants