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

ASF GitHub Bot commented on NIFI-1754:
--------------------------------------

Github user trkurc commented on a diff in the pull request:

    https://github.com/apache/nifi/pull/478#discussion_r65219394
  
    --- Diff: 
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/controller/repository/StandardProcessSession.java
 ---
    @@ -950,6 +955,45 @@ private void rollback(final boolean penalize, final 
boolean rollbackCheckpoint)
             resetState();
         }
     
    +    private String loggableFlowfileInfo() {
    +        final StringBuilder details = new StringBuilder(1024).append("[");
    +        final int initLen = details.length();
    +        int filesListed = 0;
    +        int maxRollbackFlowfilesToLog = 5;
    --- End diff --
    
    @jskora let me know out of band that this is to silence an IDE 
"notification" about the constant only being used in one spot


> Rollback log messages should include the flowfile filename and UUID to assist 
> in flow management.
> -------------------------------------------------------------------------------------------------
>
>                 Key: NIFI-1754
>                 URL: https://issues.apache.org/jira/browse/NIFI-1754
>             Project: Apache NiFi
>          Issue Type: Improvement
>          Components: Core Framework
>    Affects Versions: 1.0.0, 0.7.0
>            Reporter: Joe Skora
>            Assignee: Joe Skora
>
> When a processor calls rollback on a flowfile, the filename and UUID of the 
> flowfile should be included in log entry to make it easier for the offending 
> files to be found and fixed or removed from the flow by the dataflow managers.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to