This is an automated email from the ASF dual-hosted git repository.
exceptionfactory pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/nifi.git
The following commit(s) were added to refs/heads/main by this push:
new 2b83fe91430 NIFI-16127 Removed redundant usage of unmodifiable
Collection wrappers (#11448)
2b83fe91430 is described below
commit 2b83fe9143063abd2ccb6b5ca2d5b4649177b2d5
Author: dan-s1 <[email protected]>
AuthorDate: Mon Jul 20 17:20:27 2026 -0400
NIFI-16127 Removed redundant usage of unmodifiable Collection wrappers
(#11448)
Signed-off-by: David Handermann <[email protected]>
---
.../nifi/processors/script/engine/ClojureScriptEngineFactory.java | 4 ++--
.../apache/nifi/controller/reporting/AbstractReportingContext.java | 2 +-
.../src/main/java/org/apache/nifi/util/FlowInfo.java | 3 +--
3 files changed, 4 insertions(+), 5 deletions(-)
diff --git
a/nifi-extension-bundles/nifi-scripting-bundle/nifi-scripting-processors/src/main/java/org/apache/nifi/processors/script/engine/ClojureScriptEngineFactory.java
b/nifi-extension-bundles/nifi-scripting-bundle/nifi-scripting-processors/src/main/java/org/apache/nifi/processors/script/engine/ClojureScriptEngineFactory.java
index dd3c9e00140..eba6bb5d32e 100644
---
a/nifi-extension-bundles/nifi-scripting-bundle/nifi-scripting-processors/src/main/java/org/apache/nifi/processors/script/engine/ClojureScriptEngineFactory.java
+++
b/nifi-extension-bundles/nifi-scripting-bundle/nifi-scripting-processors/src/main/java/org/apache/nifi/processors/script/engine/ClojureScriptEngineFactory.java
@@ -28,9 +28,9 @@ import javax.script.ScriptEngineFactory;
*/
public class ClojureScriptEngineFactory implements ScriptEngineFactory {
- public static final List<String> EXTENSIONS =
Collections.unmodifiableList(Collections.singletonList("clj"));
+ public static final List<String> EXTENSIONS =
Collections.singletonList("clj");
public static final List<String> MIME_TYPES =
Collections.unmodifiableList(Arrays.asList("application/clojure",
"text/clojure"));
- public static final List<String> NAMES =
Collections.unmodifiableList(Collections.singletonList("Clojure"));
+ public static final List<String> NAMES =
Collections.singletonList("Clojure");
// This engine provides constants, global engine properties, etc. to be
returned by the ScriptEngineFactory interface methods,
// and is not used to execute scripts. A new ClojureScriptEngine will be
returned by each call to getScriptEngine()
diff --git
a/nifi-framework-bundle/nifi-framework/nifi-framework-components/src/main/java/org/apache/nifi/controller/reporting/AbstractReportingContext.java
b/nifi-framework-bundle/nifi-framework/nifi-framework-components/src/main/java/org/apache/nifi/controller/reporting/AbstractReportingContext.java
index b67d1b7de7b..a19238e3828 100644
---
a/nifi-framework-bundle/nifi-framework/nifi-framework-components/src/main/java/org/apache/nifi/controller/reporting/AbstractReportingContext.java
+++
b/nifi-framework-bundle/nifi-framework/nifi-framework-components/src/main/java/org/apache/nifi/controller/reporting/AbstractReportingContext.java
@@ -84,7 +84,7 @@ public abstract class AbstractReportingContext implements
ReportingContext {
@Override
public Map<PropertyDescriptor, String> getProperties() {
- return Collections.unmodifiableMap(properties);
+ return properties;
}
@Override
diff --git
a/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/util/FlowInfo.java
b/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/util/FlowInfo.java
index 9d6f3b79691..444864a071e 100644
---
a/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/util/FlowInfo.java
+++
b/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/util/FlowInfo.java
@@ -30,8 +30,7 @@ public class FlowInfo {
public FlowInfo(final String rootGroupId, final List<PortDTO> ports) {
this.rootGroupId = rootGroupId;
- this.ports = (ports == null ?
Collections.unmodifiableList(Collections.EMPTY_LIST) :
- Collections.unmodifiableList(new ArrayList<>(ports)));
+ this.ports = (ports == null ? List.of() :
Collections.unmodifiableList(new ArrayList<>(ports)));
}
public String getRootGroupId() {
