David Turner

Run ESAllocationTestCase against both allocators

1

Today the unit tests on the desired balance allocator feature branch still default to using the `BalancedShardsAllocator`. With this commit we randomly choose between `BalancedShardsAllocator` and `DesiredBalanceShardsAllocator`.

Fail writing snapshot data to filesystem repo if space running out

9

A user [reported running out of disk space](https://discuss.elastic.co/t/unable-to-delete-snapshot-no-space-left-on-device-solved/261675) in their shared filesystem repository which left it completely stuck, unable to take any further actions since everything that might delete any...

[CI] InternalEngineTests testMergeThreadLogging failing

1

Also seen in a local run of `./gradlew :server:test` but doesn't easily reproduce, even when running many iterations in a loop. **Build scan:** https://gradle-enterprise.elastic.co/s/kbbpmnbim32ck/tests/:server:test/org.elasticsearch.index.engine.InternalEngineTests/testMergeThreadLogging **Reproduction line:** `./gradlew ':server:test' --tests "org.elasticsearch.index.engine.InternalEngineTests.testMergeThreadLogging"...

Today the master's pending task queue is just the `PriorityBlockingQueue` belonging to the underlying `ThreadPoolExecutor`. The reasons for this date back a long way but it doesn't really reflect the...

Creating a snapshot does not verify that all nodes are writing to the same blobstore

1

Snapshots work by writing to a blobstore in which the same blob can be accessed at the same path across all nodes. By default we check that the blobstore is...

IndicesClusterStateService does (sometimes substantial amounts of) IO on cluster applier thread

1

When `IndicesClusterStateService` applies a new cluster state it does substantial amounts of work inline on the applier thread, including some IO (or blocking waits for IO on other threads) which...

Binding to the first free port in a given range is trappy

6

Today `transport.port` and `http.port` accept a range of ports. At startup, Elasticsearch binds to the first available port in the range. This is useful for testing because it lets you...

Please could you add this library to stackage?

2

Thanks.

`URLRepository` should not block shutdown

2

Today a node with a registered `URLRepository` will not shut down cleanly because it never releases the last of the `activityRefs`. This commit fixes that.

Protect get-snapshots APIs against excessive memory use

2

The get-snapshots APIs accumulate the details of all matching snapshots in memory before responding. In a high-index-count cluster with a long retention period on snapshots this can add up to...