You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nifi.apache.org by ex...@apache.org on 2026/07/20 21:34:40 UTC

(nifi) branch main updated: NIFI-16128 Removed redundant calls to String.format (#11449)

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 648fca5c81b NIFI-16128 Removed redundant calls to String.format (#11449)
648fca5c81b is described below

commit 648fca5c81b35541e28536aafdf34ea0160e5629
Author: dan-s1 <ds...@gmail.com>
AuthorDate: Mon Jul 20 17:34:34 2026 -0400

    NIFI-16128 Removed redundant calls to String.format (#11449)
    
    Signed-off-by: David Handermann <ex...@apache.org>
---
 .../java/org/apache/nifi/processors/aws/dynamodb/GetDynamoDB.java   | 6 +++---
 .../org/apache/nifi/web/StandardNiFiWebConfigurationContext.java    | 6 +++---
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/nifi-extension-bundles/nifi-aws-bundle/nifi-aws-processors/src/main/java/org/apache/nifi/processors/aws/dynamodb/GetDynamoDB.java b/nifi-extension-bundles/nifi-aws-bundle/nifi-aws-processors/src/main/java/org/apache/nifi/processors/aws/dynamodb/GetDynamoDB.java
index 700e3d5821e..a7973ff228e 100644
--- a/nifi-extension-bundles/nifi-aws-bundle/nifi-aws-processors/src/main/java/org/apache/nifi/processors/aws/dynamodb/GetDynamoDB.java
+++ b/nifi-extension-bundles/nifi-aws-bundle/nifi-aws-processors/src/main/java/org/apache/nifi/processors/aws/dynamodb/GetDynamoDB.java
@@ -138,7 +138,7 @@ public class GetDynamoDB extends AbstractDynamoDBProcessor {
             results.add(new ConfigVerificationResult.Builder()
                     .outcome(Outcome.SUCCESSFUL)
                     .verificationStepName("Configure DynamoDB BatchGetItems Request")
-                    .explanation(String.format("Successfully configured BatchGetItems Request"))
+                    .explanation("Successfully configured BatchGetItems Request")
                     .build());
         } catch (final IllegalArgumentException e) {
             verificationLogger.error("Failed to configured BatchGetItems Request", e);
@@ -154,7 +154,7 @@ public class GetDynamoDB extends AbstractDynamoDBProcessor {
             results.add(new ConfigVerificationResult.Builder()
                     .outcome(Outcome.SKIPPED)
                     .verificationStepName("Get DynamoDB Items")
-                    .explanation(String.format("Skipped getting DynamoDB items because no primary keys would be included in retrieval"))
+                    .explanation("Skipped getting DynamoDB items because no primary keys would be included in retrieval")
                     .build());
         } else {
             try {
@@ -168,7 +168,7 @@ public class GetDynamoDB extends AbstractDynamoDBProcessor {
                     results.add(new ConfigVerificationResult.Builder()
                             .outcome(Outcome.SUCCESSFUL)
                             .verificationStepName("Get DynamoDB Items")
-                            .explanation(String.format("Successfully issued request, although no items were returned from DynamoDB"))
+                            .explanation("Successfully issued request, although no items were returned from DynamoDB")
                             .build());
                 } else {
                     // Handle processed items and get the json document
diff --git a/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/StandardNiFiWebConfigurationContext.java b/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/StandardNiFiWebConfigurationContext.java
index f3eb8bb81f3..208a0425612 100644
--- a/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/StandardNiFiWebConfigurationContext.java
+++ b/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/StandardNiFiWebConfigurationContext.java
@@ -231,7 +231,7 @@ public class StandardNiFiWebConfigurationContext implements NiFiWebConfiguration
         final String id = requestContext.getId();
 
         if (StringUtils.isBlank(id)) {
-            throw new ResourceNotFoundException(String.format("Configuration request context config did not have a component ID."));
+            throw new ResourceNotFoundException("Configuration request context config did not have a component ID.");
         }
 
         // ensure the path could be
@@ -263,7 +263,7 @@ public class StandardNiFiWebConfigurationContext implements NiFiWebConfiguration
         final String id = requestContext.getId();
 
         if (StringUtils.isBlank(id)) {
-            throw new ResourceNotFoundException(String.format("Configuration request context did not have a component ID."));
+            throw new ResourceNotFoundException("Configuration request context did not have a component ID.");
         }
 
         // ensure the path could be
@@ -1134,7 +1134,7 @@ public class StandardNiFiWebConfigurationContext implements NiFiWebConfiguration
             throw new ClusterRequestException(nodeResponse.getThrowable());
         } else if (nodeResponse.getClientResponse().getStatus() == Response.Status.CONFLICT.getStatusCode()) {
             ClientResponseUtils.drainClientResponse(nodeResponse.getClientResponse());
-            throw new InvalidRevisionException(String.format("NiFi is unable to process the request at this time."));
+            throw new InvalidRevisionException("NiFi is unable to process the request at this time.");
         } else if (nodeResponse.getClientResponse().getStatus() == Response.Status.NOT_FOUND.getStatusCode()) {
             ClientResponseUtils.drainClientResponse(nodeResponse.getClientResponse());
             throw new ResourceNotFoundException("Unable to find component with id: " + id);