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

    https://github.com/apache/nifi/pull/2751#discussion_r194034751
  
    --- Diff: 
nifi-nar-bundles/nifi-aws-bundle/nifi-aws-processors/src/main/java/org/apache/nifi/processors/aws/s3/PutS3Object.java
 ---
    @@ -205,11 +210,21 @@
                 .defaultValue(NO_SERVER_SIDE_ENCRYPTION)
                 .build();
     
    +    public static final PropertyDescriptor OBJECT_TAGS = new 
PropertyDescriptor.Builder()
    --- End diff --
    
    Yep, wouldn't it be more flexible?
    With the current approach, it assumes that you always have the same set of 
tags (even if the values can change based on ffs attributes, the keys would 
always be the same). With the regular expression, you could easily manage the 
case where you have one ff with attributes tagS3_country=FR, 
tagS3_security=topsecret and one flow file with only tagS3_country=US. You 
would set the tag regular expression to tagS3.* and you would have the tags 
created using the attributes matching the regex. Does it make sense or am I 
missing something?


---

Reply via email to