From 8075bb6a76d63c635762f8d741bd05d975227bce Mon Sep 17 00:00:00 2001
From: Henning Andersen <henning.andersen@elastic.co>
Date: Mon, 5 Jul 2021 14:26:33 +0200
Subject: [PATCH] Less RoutingNodes allocations

No longer create as many RoutingNodes, since they are expensive.

Closes #74325
---
 .../autoscaling/storage/ReactiveStorageDeciderService.java  | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/x-pack/plugin/autoscaling/src/main/java/org/elasticsearch/xpack/autoscaling/storage/ReactiveStorageDeciderService.java b/x-pack/plugin/autoscaling/src/main/java/org/elasticsearch/xpack/autoscaling/storage/ReactiveStorageDeciderService.java
index 1f866fcef3099..c9bb84f1f516f 100644
--- a/x-pack/plugin/autoscaling/src/main/java/org/elasticsearch/xpack/autoscaling/storage/ReactiveStorageDeciderService.java
+++ b/x-pack/plugin/autoscaling/src/main/java/org/elasticsearch/xpack/autoscaling/storage/ReactiveStorageDeciderService.java
@@ -224,10 +224,9 @@ public static class AllocationState {
         }
 
         public long storagePreventsAllocation() {
-            RoutingNodes routingNodes = new RoutingNodes(state, false);
             RoutingAllocation allocation = new RoutingAllocation(
                 allocationDeciders,
-                routingNodes,
+                state.getRoutingNodes(),
                 state,
                 info,
                 shardSizeInfo,
@@ -241,10 +240,9 @@ public long storagePreventsAllocation() {
         }
 
         public long storagePreventsRemainOrMove() {
-            RoutingNodes routingNodes = new RoutingNodes(state, false);
             RoutingAllocation allocation = new RoutingAllocation(
                 allocationDeciders,
-                routingNodes,
+                state.getRoutingNodes(),
                 state,
                 info,
                 shardSizeInfo,