[GitHub] [nifi] exceptionfactory commented on a diff in pull request #6540: NIFI-10656 Log ignored event with info instead of warning

[GitBox <gi...@apache.org>]

exceptionfactory commented on code in PR #6540:
URL: https://github.com/apache/nifi/pull/6540#discussion_r997156309


##########
nifi-nar-bundles/nifi-azure-bundle/nifi-azure-processors/src/main/java/org/apache/nifi/processors/azure/storage/PutAzureDataLakeStorage.java:
##########
@@ -161,10 +161,10 @@ public void onTrigger(final ProcessContext context, final ProcessSession session
                 if (dlsException.getStatusCode() == 409) {
                     if (conflictResolution.equals(IGNORE_RESOLUTION)) {
                         session.transfer(flowFile, REL_SUCCESS);
-                        String warningMessage = String.format("File with the same name already exists. " +
+                        final String logMessage = String.format("File with the same name already exists. " +
                                 "Remote file not modified. " +
                                 "Transferring {} to success due to %s being set to '%s'.", CONFLICT_RESOLUTION.getDisplayName(), conflictResolution);
-                        getLogger().warn(warningMessage, new Object[]{flowFile});
+                        getLogger().info(logMessage, flowFile);

Review Comment:
   The approach should be adjusted to use `{}` placeholder arguments instead of `String.format()`.
   
   



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@nifi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org