ASF GitHub Bot commented on NIFI-4637:

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

    --- Diff: 
    @@ -103,6 +103,17 @@
    +    static final PropertyDescriptor VISIBLITY_LABEL = new 
    +            .name("delete-visibility-label")
    +            .displayName("Visibility Label")
    +            .description("If visibility labels are enabled, a row cannot 
be deleted without supplying its visibility label(s) in the delete " +
    +                    "request. Note: this visibility label will be applied 
to all cells within the row that is specified. If some cells have " +
    +                    "different visibility labels, they will not be 
deleted. When that happens, the failure to delete will be considered a success 
" +
    +                    "because HBase does not report it as a failure.")
    +            .required(false)
    --- End diff --
    Since this is not required and always valid, a user could force the 
property to be set with an empty string. Is that OK? It means visibility would 
be equal to "". Does that make sense in some cases? Wouldn't be better to add a 
non-empty string validator?

> Add support for HBase visibility labels to HBase processors and controller 
> services
> -----------------------------------------------------------------------------------
>                 Key: NIFI-4637
>                 URL: https://issues.apache.org/jira/browse/NIFI-4637
>             Project: Apache NiFi
>          Issue Type: Improvement
>            Reporter: Mike Thomsen
>            Assignee: Mike Thomsen
>            Priority: Major
> HBase supports visibility labels, but you can't use them from NiFi because 
> there is no way to set them. The existing processors and services should be 
> upgraded to handle this capability.

This message was sent by Atlassian JIRA

Reply via email to