accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Joe Ferner" <joe.m.fer...@gmail.com>
Subject Re: Review Request 18917: Add a NOT (!) operator to ColumnVisibility (ACCUMULO-2439)
Date Fri, 07 Mar 2014 19:41:53 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18917/
-----------------------------------------------------------

(Updated March 7, 2014, 7:41 p.m.)


Review request for accumulo.


Repository: accumulo


Description (updated)
-------

This patch adds a NOT "!" operator to ColumnVisibility.

The syntax is as follows:
!a
(!a)&(!b)
a&(!b)
a&(!(b|c))

Because of the nature of the current visibility parsing algorithm the additional parentheses
are required.
In the shell, the "Unable to render embedded object: File (" requires escaping. This is due
to how JLine parses the command and attempts to substitute ") not found." with history.


Diffs
-----

  core/src/main/java/org/apache/accumulo/core/security/ColumnVisibility.java 75091d2 
  core/src/main/java/org/apache/accumulo/core/security/VisibilityEvaluator.java 725b2c7 
  core/src/test/java/org/apache/accumulo/core/security/ColumnVisibilityTest.java 7a6a80d 
  core/src/test/java/org/apache/accumulo/core/security/VisibilityEvaluatorTest.java ee4d2ee


Diff: https://reviews.apache.org/r/18917/diff/


Testing (updated)
-------

This patch includes unit tests for parsing, flattening, and evaluating the not operator.


Thanks,

Joe Ferner


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message