-Re: Review Request 18917: Add a NOT (!) operator to ColumnVisibility (ACCUMULO-2439)
This is an automatically generated e-mail. To reply, visit:
The patch should make this feature optional. Preferably, make the visibility evaluation pluggable. At the very least, there's good reasons to disallow NOT terms, so this shouldn't be the default.
I'd also like to see some additional tests, that read and write data to verify the feature end-to-end. This is changing some very important code that's been stable for some time, so more tests are always better.
- Christopher Tubbs
On March 7, 2014, 2:41 p.m., Joe Ferner wrote: