jfrazee commented on a change in pull request #5098:
URL: https://github.com/apache/nifi/pull/5098#discussion_r657442804



##########
File path: 
nifi-nar-bundles/nifi-azure-bundle/nifi-azure-processors/src/main/java/org/apache/nifi/processors/azure/storage/ListAzureDataLakeStorage.java
##########
@@ -129,6 +131,16 @@
             
.expressionLanguageSupported(ExpressionLanguageScope.VARIABLE_REGISTRY)
             .build();
 
+    public static final PropertyDescriptor TEMP_FILE_PREFIX = new 
PropertyDescriptor.Builder()
+            .name("azure-temp-file-prefix")
+            .displayName("Temp File Prefix for Azure")

Review comment:
       > @jfrazee may have a better understanding of the runtime environment, 
but do temporary files always follow a standard pattern? If so, perhaps having 
a Boolean property for filtering temporary files would be easier to understand 
and provide the desired behavior.
   
   I may not have been thinking clearly because I was thinking of files like 
_SUCCESS and _temporary which are used more like dotted files in HDFS. I think 
it'll be unlikely to see those though unless someone just did a bulk transfer 
from HDFS to ADLS so it's probably a non-concern unless someone says they need 
it.




-- 
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.

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


Reply via email to