Skip to content

Conversation

KShivendu
Copy link
Member

@KShivendu KShivendu commented Jul 4, 2025

Applied suggestion from #6800 (comment)

All Submissions:

  • Contributions should target the dev branch. Did you create your branch from dev?
  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

@KShivendu KShivendu force-pushed the fix-resharding-dead-replicas branch from 9bdc13d to 50b476a Compare July 4, 2025 12:08
@timvisee timvisee merged commit 3cdc2b1 into fix-resharding-dead-replicas Jul 15, 2025
19 checks passed
@timvisee timvisee deleted the remove-is-in-progress branch July 15, 2025 13:38
@KShivendu KShivendu restored the remove-is-in-progress branch July 15, 2025 14:15
@KShivendu KShivendu deleted the remove-is-in-progress branch July 15, 2025 14:16
KShivendu added a commit that referenced this pull request Jul 16, 2025
…esharding (#6800)

* Fix bug that causes all replicas to die if node is restarted during resharding

* fix recursive async problem

* Avoid write lock unless required

* avoid using &mut when & is sufficient

* Remove is_in_progress since check_abort_resharding exists (#6806)

* On resharding abort, only abort transfers related to current operation

* Fix resharding dead replicas improvements (#6881)

* Apply suggestions to fix resharding dead replicas

* fmt

---------

Co-authored-by: timvisee <tim@visee.me>
generall pushed a commit that referenced this pull request Jul 17, 2025
…esharding (#6800)

* Fix bug that causes all replicas to die if node is restarted during resharding

* fix recursive async problem

* Avoid write lock unless required

* avoid using &mut when & is sufficient

* Remove is_in_progress since check_abort_resharding exists (#6806)

* On resharding abort, only abort transfers related to current operation

* Fix resharding dead replicas improvements (#6881)

* Apply suggestions to fix resharding dead replicas

* fmt

---------

Co-authored-by: timvisee <tim@visee.me>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants