Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
Accumulo, mail # dev - Review Request 18917: Add a NOT (!) operator to ColumnVisibility (ACCUMULO-2439)


Copy link to this message
-
Re: Review Request 18917: Add a NOT (!) operator to ColumnVisibility (ACCUMULO-2439)
Christopher Tubbs 2014-03-07, 21:19

This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18917/#review36575
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: