Skip to content

Revert "Attempt to clean up index before remote transfer (#115142)"#139569

Merged
lkts merged 4 commits intoelastic:mainfrom
lkts:revert_115142
Jan 5, 2026
Merged

Revert "Attempt to clean up index before remote transfer (#115142)"#139569
lkts merged 4 commits intoelastic:mainfrom
lkts:revert_115142

Conversation

@lkts
Copy link
Contributor

@lkts lkts commented Dec 15, 2025

This reverts commit fc67f7c.

See #117249 (comment). It was also previously suggested it causes #117024. Given that this is an optimization that is not required for correctness i opted to revert it and re-introduce if needed.

@lkts lkts added >bug :Distributed/Recovery Anything around constructing a new shard, either from a local or a remote source. labels Dec 31, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@lkts lkts marked this pull request as ready for review January 2, 2026 18:04
@elasticsearchmachine elasticsearchmachine added the Team:Distributed Indexing (obsolete) Meta label for Distributed Indexing team. Obsolete. Please do not use. label Jan 2, 2026
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-distributed-indexing (Team:Distributed Indexing)

Copy link
Contributor

@bcully bcully left a comment

Choose a reason for hiding this comment

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

LGTM thank you!

@lkts lkts merged commit 4a0d00b into elastic:main Jan 5, 2026
35 checks passed
@lkts lkts deleted the revert_115142 branch January 5, 2026 16:56
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
9.1
9.2
8.19
9.3
lkts added a commit to lkts/elasticsearch that referenced this pull request Jan 5, 2026
)" (elastic#139569)

* Revert "Attempt to clean up index before remote transfer (elastic#115142)"

This reverts commit fc67f7c.
lkts added a commit to lkts/elasticsearch that referenced this pull request Jan 5, 2026
)" (elastic#139569)

* Revert "Attempt to clean up index before remote transfer (elastic#115142)"

This reverts commit fc67f7c.
lkts added a commit to lkts/elasticsearch that referenced this pull request Jan 5, 2026
)" (elastic#139569)

* Revert "Attempt to clean up index before remote transfer (elastic#115142)"

This reverts commit fc67f7c.
elasticsearchmachine pushed a commit that referenced this pull request Jan 5, 2026
…139569) (#140180)

* Revert "Attempt to clean up index before remote transfer (#115142)"

This reverts commit fc67f7c.
elasticsearchmachine pushed a commit that referenced this pull request Jan 5, 2026
…139569) (#140179)

* Revert "Attempt to clean up index before remote transfer (#115142)"

This reverts commit fc67f7c.
elasticsearchmachine pushed a commit that referenced this pull request Jan 5, 2026
…139569) (#140181)

* Revert "Attempt to clean up index before remote transfer (#115142)"

This reverts commit fc67f7c.
lkts added a commit that referenced this pull request Jan 5, 2026
…139569) (#140178)

* Revert "Attempt to clean up index before remote transfer (#115142)"

This reverts commit fc67f7c.
lkts added a commit to lkts/elasticsearch that referenced this pull request Jan 7, 2026
lkts added a commit that referenced this pull request Jan 8, 2026
lkts added a commit to lkts/elasticsearch that referenced this pull request Jan 8, 2026
lkts added a commit to lkts/elasticsearch that referenced this pull request Jan 9, 2026
(cherry picked from commit fc4efde)

# Conflicts:
#	muted-tests.yml
jimczi pushed a commit to jimczi/elasticsearch that referenced this pull request Jan 12, 2026
lkts added a commit to lkts/elasticsearch that referenced this pull request Jan 12, 2026
lkts added a commit that referenced this pull request Jan 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged >bug :Distributed/Recovery Anything around constructing a new shard, either from a local or a remote source. Team:Distributed Indexing (obsolete) Meta label for Distributed Indexing team. Obsolete. Please do not use. v8.19.10 v9.1.10 v9.2.4 v9.3.1 v9.4.0

3 participants