Removes redundent NORELEASES and ObjectParserUtils (#32427)

Author: colings86

x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/indexlifecycle/IndexLifecycleMetadata.java

@@ -38,10 +38,11 @@ public class IndexLifecycleMetadata implements XPackMetaDataCustom {
     public static final IndexLifecycleMetadata EMPTY = new IndexLifecycleMetadata(Collections.emptySortedMap(), OperationMode.RUNNING);
 
     @SuppressWarnings("unchecked")
-    public static final ConstructingObjectParser<IndexLifecycleMetadata, Void> PARSER = new ConstructingObjectParser<>(
-            TYPE, a -> new IndexLifecycleMetadata(
-                    ObjectParserUtils.convertListToMapValues(LifecyclePolicyMetadata::getName, (List<LifecyclePolicyMetadata>) a[0]),
-                        OperationMode.valueOf((String) a[1])));
+    public static final ConstructingObjectParser<IndexLifecycleMetadata, Void> PARSER = new ConstructingObjectParser<>(TYPE,
+            a -> new IndexLifecycleMetadata(
+                    ((List<LifecyclePolicyMetadata>) a[0]).stream()
+                            .collect(Collectors.toMap(LifecyclePolicyMetadata::getName, Function.identity())),
+                    OperationMode.valueOf((String) a[1])));
     static {
         PARSER.declareNamedObjects(ConstructingObjectParser.constructorArg(), (p, c, n) -> LifecyclePolicyMetadata.parse(p, n),
                 v -> {

x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/indexlifecycle/ObjectParserUtils.java

@@ -22,8 +22,8 @@
 import java.util.Map;
 import java.util.Objects;
 import java.util.TreeMap;
-
-import static org.elasticsearch.xpack.core.indexlifecycle.ObjectParserUtils.convertListToMapValues;
+import java.util.function.Function;
+import java.util.stream.Collectors;
 
 /**
  * Represents set of {@link LifecycleAction}s which should be executed at a
@@ -36,8 +36,8 @@ public class Phase implements ToXContentObject, Writeable {
 
     @SuppressWarnings("unchecked")
     private static final ConstructingObjectParser<Phase, String> PARSER = new ConstructingObjectParser<>("phase", false,
-            (a, name) -> new Phase(name, (TimeValue) a[0],
-        convertListToMapValues(LifecycleAction::getWriteableName, (List<LifecycleAction>) a[1])));
+            (a, name) -> new Phase(name, (TimeValue) a[0], ((List<LifecycleAction>) a[1]).stream()
+                    .collect(Collectors.toMap(LifecycleAction::getWriteableName, Function.identity()))));
     static {
         PARSER.declareField(ConstructingObjectParser.optionalConstructorArg(),
                 (p, c) -> TimeValue.parseTimeValue(p.text(), AFTER_FIELD.getPreferredName()), AFTER_FIELD, ValueType.VALUE);

x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/indexlifecycle/Phase.java

@@ -28,8 +28,7 @@ public class LifecyclePolicyMetadataTests extends AbstractSerializingTestCase<Li
 
     @Before
     public void setup() {
-        lifecycleName = randomAlphaOfLength(20); // NORELEASE we need to randomise the lifecycle name rather
-                                                 // than use the same name for all instances
+        lifecycleName = randomAlphaOfLength(20);
     }
 
     @Override

x-pack/plugin/core/src/test/java/org/elasticsearch/xpack/core/indexlifecycle/LifecyclePolicyMetadataTests.java

@@ -28,8 +28,7 @@ public class PhaseTests extends AbstractSerializingTestCase<Phase> {
 
     @Before
     public void setup() {
-        phaseName = randomAlphaOfLength(20); // NORELEASE we need to randomise the phase name rather 
-                                             // than use the same name for all instances
+        phaseName = randomAlphaOfLength(20);
     }
 
     @Override

x-pack/plugin/core/src/test/java/org/elasticsearch/xpack/core/indexlifecycle/PhaseTests.java

@@ -34,8 +34,7 @@ public class PutLifecycleRequestTests extends AbstractStreamableXContentTestCase
 
     @Before
     public void setup() {
-        lifecycleName = randomAlphaOfLength(20); // NORELEASE we need to randomise the lifecycle name rather 
-                                                 // than use the same name for all instances
+        lifecycleName = randomAlphaOfLength(20);
     }
 
     @Override