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

    https://github.com/apache/nifi/pull/2518#discussion_r184277181
  
    --- Diff: 
nifi-nar-bundles/nifi-hbase-bundle/nifi-hbase-processors/src/main/java/org/apache/nifi/hbase/PutHBaseRecord.java
 ---
    @@ -142,6 +161,16 @@
                 .allowableValues(NULL_FIELD_EMPTY, NULL_FIELD_SKIP)
                 .build();
     
    +    protected static final PropertyDescriptor VISIBILITY_RECORD_PATH = new 
PropertyDescriptor.Builder()
    +            .name("put-hb-rec-visibility-record-path")
    +            .displayName("Visibility String Record Path Root")
    +            .description("A record path that points to part of the record 
which contains a path to a mapping of visibility strings to record paths")
    +            .required(false)
    +            .addValidator(Validator.VALID)
    +            .build();
    --- End diff --
    
    I like this configuration. Should we add similar property to PutHBaseJson, 
in order to pass visibility label within the input JSON tree? Similar to 
`Column Family` property. How do you think?


---

Reply via email to