[GitHub] nifi pull request #2733: NIFI-5228: Allow user to choose whether or not to a...

[mgaido91]

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

    https://github.com/apache/nifi/pull/2733#discussion_r190156015
  
    --- Diff: 
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/ListFile.java
 ---
    @@ -255,43 +262,47 @@ public void onScheduled(final ProcessContext context) 
{
             final Path absPath = filePath.toAbsolutePath();
             final String absPathString = absPath.getParent().toString() + 
File.separator;
     
    +        final DateFormat formatter = new 
SimpleDateFormat(FILE_MODIFY_DATE_ATTR_FORMAT, Locale.US);
    --- End diff --
    
    nit: what about creating a `ThreadLocal` global variable for this, in order 
to avoid creating one instance for each item?


---