David Turner

Stricter failure handling in `TransportGetSnapshotsAction`

15

Today if there's a failure during a multi-repo get-snapshots request then we record a per-repository failure but allow the rest of the request to proceed. This is trappy for clients,...

High heap usage due to snapshot post-deletion cleanup

1

When deleting a snapshot we accumulate in memory a list of all the blobs that can be deleted after the repository update is committed. Each blob name takes only ~80B...

More discriminating `RESTART` shutdown logic

3

In a rolling restart we recommend users wait for the cluster health to reach `green` in between node restarts, and some users will also wait for rebalancing to complete each...

TransportMasterNodeAction may use stale cluster state for routing to current master

4

Different nodes may have different beliefs about the identity of the current elected master, and in particular there can be times when two nodes each think the other is the...

Fix timeouts in Datastream Lifecycle module

1

Relates #107984

Use `scheduleUnlessShuttingDown` in `LeaderChecker`

2

We mustn't throw an exception here, it trips an assertion. Simply giving up on shutdown is fine. Closes #108642

[CI] `EsRejectedExecutionException` trips `TransportService` assertion on shutdown

1

Not directly related to this test I think. **Build scan:** https://gradle-enterprise.elastic.co/s/msnan5s2tjtxy/tests/:server:internalClusterTest/org.elasticsearch.action.admin.indices.refresh.RefreshBlocksIT/testRefreshWithBlocks **Reproduction line:** ``` ./gradlew ':server:internalClusterTest' --tests "org.elasticsearch.action.admin.indices.refresh.RefreshBlocksIT.testRefreshWithBlocks" -Dtests.seed=9FA63D5B374741D3 -Dtests.locale=ko -Dtests.timezone=America/Yakutat -Druntime.java=21 ``` **Applicable branches:** main **Reproduces locally?:** Didn't try...

Use `scheduleUnlessShuttingDown` in `FollowersChecker`

1

Relates #108643

Remove TODOs from ES-8334

1

Upon further discussion we decided not to do any of these things, so this commit removes the leftover TODO comments.

Add `?master_timeout` to `RestDeprecationInfoAction`

2

Relates #107984.