[ 
https://issues.apache.org/jira/browse/NIFI-13265?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17850430#comment-17850430
 ] 

Daniel Stieglitz commented on NIFI-13265:
-----------------------------------------

[~exceptionfactory] Thanks for clarifying. How about for something like this in 
PutS3Object lines 925-929

{code:java}
getLogger().warn("AccessDenied checking S3 Multipart Upload list for {}: {} " +
                            "** The configured user does not have the 
s3:ListBucketMultipartUploads permission " +
                            "for this bucket, S3 ageoff cannot occur without 
this permission.  Next ageoff check " +
                            "time is being advanced by interval to prevent 
checking on every upload **",
                            new Object[]{bucket, e.getMessage()});
{code}

should i completely remove the parameter for the exception as 

{code:java}
getLogger().warn("AccessDenied checking S3 Multipart Upload list for {}: " +
                            "** The configured user does not have the 
s3:ListBucketMultipartUploads permission " +
                            "for this bucket, S3 ageoff cannot occur without 
this permission.  Next ageoff check " +
                            "time is being advanced by interval to prevent 
checking on every upload **", bucket, e);
{code}




> Remove the instantiation of Object arrays for arguments in ComponentLog log 
> and org.slf4j.Logger statements
> -----------------------------------------------------------------------------------------------------------
>
>                 Key: NIFI-13265
>                 URL: https://issues.apache.org/jira/browse/NIFI-13265
>             Project: Apache NiFi
>          Issue Type: Improvement
>            Reporter: Daniel Stieglitz
>            Assignee: Daniel Stieglitz
>            Priority: Minor
>
> There are still classes after the changes in NIFI-12075 and NIFI-12076 which 
> instantiate an Object array for ComponentLog log statements. This ticket aims 
> to remove those instantiations. In addition similar changes should be made to 
> org.slf4j.Logger statements.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to