New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[8.13] [Test] Ranged read should read non-empty content (#106000) #106525

Merged

elasticsearchmachine merged 1 commit into elastic:8.13 from ywangd:backport/8.13/pr-106000

Mar 20, 2024

Member

ywangd commented Mar 20, 2024

Backports the following commits to 8.13:

[Test] Ranged read should read non-empty content ([Test] Ranged read should read non-empty content #106000)


          [Test] Ranged read should read non-empty content (elastic#106000)

05800f2

Empty read is
[short-circuited](https://github.com/elastic/elasticsearch/blob/e8039b9ecb2451752ac5377c44a6a0c662087a9f/modules/repository-s3/src/main/java/org/elasticsearch/repositories/s3/S3BlobContainer.java#L115-L116)
without going to the blob store. In order to test s3 blob store, ranged
read should read at least one byte. This PR ensures that.

Resolves: elastic#105958

ywangd added :Distributed Coordination/Snapshot/Restore >test auto-merge-without-approval backport Team:Distributed (Obsolete) labels

elasticsearchmachine mentioned this pull request

[Test] Ranged read should read non-empty content #106000

Merged

elasticsearchmachine added the v8.13.1 label

elasticsearchmachine merged commit 50c71bc into elastic:8.13

14 checks passed

ywangd deleted the backport/8.13/pr-106000 branch

March 20, 2024 08:56

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

Labels

auto-merge-without-approval backport :Distributed Coordination/Snapshot/Restore Team:Distributed (Obsolete) >test v8.13.1