[ 
https://issues.apache.org/jira/browse/ACCUMULO-241?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13185104#comment-13185104
 ] 

Eric Newton commented on ACCUMULO-241:
--------------------------------------

This whitelist/blacklist discussion started because we have an immediate need 
to extend the alphabet to include ".". While all of these alternative ideas are 
great, we need "." added yesterday. A quoting mechanism, or alternative syntax 
is going to require testing and analysis of the performance which isn't 
something we can do very quickly.

We can always extend the expressions with something that is presently illegal 
like "%" and provide an alternative syntax:

{noformat}
%AND("a", OR("b", "c"), NOT("$x"))
{noformat}

But for now... I'm adding the silly "."; we can add extensibility of the column 
visibility when an actual use-case comes up.
                
> Visibility labels should blacklist non-ASCII characters instead of 
> whitelisting select ASCII characters
> -------------------------------------------------------------------------------------------------------
>
>                 Key: ACCUMULO-241
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-241
>             Project: Accumulo
>          Issue Type: Improvement
>    Affects Versions: 1.3.5
>            Reporter: John Vines
>              Labels: visibility
>             Fix For: 1.3.6
>
>
> We currently whitelist our visibility labels to only allow alphanumerics and 
> a few select delimiting characters. While we strive for human-readable 
> labels, we should instead utilize a blacklist approach where we disallow 
> parentheses, ampersands, pipes, and any non-ASCII characters. This will 
> provide users with more flexibility in labeling, while still sticking to 
> human readability.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to