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

    https://github.com/apache/nifi/pull/3000#discussion_r219435622
  
    --- Diff: 
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/util/FileTransfer.java
 ---
    @@ -141,6 +141,13 @@ default String getAbsolutePath(FlowFile flowFile, 
String remotePath) throws IOEx
             .defaultValue("false")
             .allowableValues("true", "false")
             .build();
    +    public static final PropertyDescriptor FOLLOW_SYMLINK = new 
PropertyDescriptor.Builder()
    --- End diff --
    
    Yep, you can choose a better name like just `follow-symlink`. The idea is: 
for new properties we're added to existing/new processors we want to use both 
`.name()` and `.displayName()`. The latter is what is actually displayed in the 
UI and can be changed without breaking backward compatibility while the name is 
what is used to uniquely reference the property and we cannot change the name 
without breaking existing flows.


---

Reply via email to