Skip to content

Conversation

@tlrx
Copy link
Member

@tlrx tlrx commented Jun 2, 2025

Last part size is wrongly computed to 0 when the last part's length is exactly equal to the size of a part. Would have probably be caught by an existing assertion.

Relates ES-11815

Last part size is wrongly computed to 0 when the last
part's length is exactly equal to the size of a part.

Relates ES-11815
@tlrx tlrx added >bug :Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs v9.1.0 labels Jun 2, 2025
@tlrx tlrx requested a review from henningandersen June 2, 2025 12:58
@elasticsearchmachine elasticsearchmachine added the Team:Distributed Coordination Meta label for Distributed Coordination team label Jun 2, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-distributed-coordination (Team:Distributed Coordination)

@elasticsearchmachine
Copy link
Collaborator

Hi @tlrx, I've created a changelog YAML for you.

Copy link
Contributor

@henningandersen henningandersen left a comment

Choose a reason for hiding this comment

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

LGTM.

@tlrx tlrx added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Jun 2, 2025
@elasticsearchmachine elasticsearchmachine merged commit 9b2252a into elastic:main Jun 2, 2025
18 checks passed
@tlrx tlrx deleted the 2025/06/02/ES-11815-bug-compute-last-part-size branch June 2, 2025 14:12
mridula-s109 pushed a commit that referenced this pull request Jun 2, 2025
…ads (#128746)

Last part size is wrongly computed to 0 when the last part's length is
exactly equal to the size of a part. Would have probably be caught by an
existing assertion.

Relates ES-11815
mridula-s109 pushed a commit to mridula-s109/elasticsearch that referenced this pull request Jun 3, 2025
…ads (elastic#128746)

Last part size is wrongly computed to 0 when the last part's length is
exactly equal to the size of a part. Would have probably be caught by an
existing assertion.

Relates ES-11815
joshua-adams-1 pushed a commit to joshua-adams-1/elasticsearch that referenced this pull request Jun 3, 2025
…ads (elastic#128746)

Last part size is wrongly computed to 0 when the last part's length is
exactly equal to the size of a part. Would have probably be caught by an
existing assertion.

Relates ES-11815
Samiul-TheSoccerFan pushed a commit to Samiul-TheSoccerFan/elasticsearch that referenced this pull request Jun 5, 2025
…ads (elastic#128746)

Last part size is wrongly computed to 0 when the last part's length is
exactly equal to the size of a part. Would have probably be caught by an
existing assertion.

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

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) >bug :Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs Team:Distributed Coordination Meta label for Distributed Coordination team v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants