You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@accumulo.apache.org by rweeks <gi...@git.apache.org> on 2015/08/22 08:17:56 UTC

[GitHub] accumulo pull request: Accumulo 3970

GitHub user rweeks opened a pull request:

    https://github.com/apache/accumulo/pull/43

    Accumulo 3970

    Here's a sketch of what I'm thinking for ACCUMULO-3970. It's by no means ready to be merged, I just want to send the PR as a starting point for a discussion.
    
    The idea is to define a new type of iterator, a VisibilityTransformingIterator, which can be set on a table. The iterator is only active in scan scope and is applied before the scan authorizations. Concrete subclasses of the VisibilityTransformingIterator receive individual key-value pairs that are present in the table and, for each pair, produce zero or more extra representations of that value. How these extra representations are produced will probably be driven by how an organization wants to anonymize or de-identify its data.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/phemisystems/accumulo ACCUMULO-3970

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/accumulo/pull/43.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #43
    
----
commit 646974ebe519246ea5340f05211972a8ae951543
Author: Russ Weeks <rw...@phemi.com>
Date:   2015-08-15T04:53:51Z

    Added test for VisibilityTransformingIterator

commit e6376c8011346c9ef8eb3d0ac762f504a6fa42e3
Author: Russ Weeks <rw...@newbrightidea.com>
Date:   2015-08-16T03:49:44Z

    Unit tests passing

commit a72ed29175970158b216d0233aba39eff8cb4c0a
Author: Russ Weeks <rw...@newbrightidea.com>
Date:   2015-08-21T18:48:22Z

    Merge remote-tracking branch 'origin/master' into vti

commit b4d89da57603dff4f8db216c8835948c21c6d689
Author: Russ Weeks <rw...@newbrightidea.com>
Date:   2015-08-22T06:13:01Z

    Fixing my new Property definition. Seems like a CLASSNAME type property can't have a default value of null

commit 06718e203fe7fd22d4b8639ff3b76382b2090333
Author: Russ Weeks <rw...@newbrightidea.com>
Date:   2015-08-22T06:14:23Z

    Removing some commented-out code

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] accumulo pull request: Accumulo 3970

Posted by ctubbsii <gi...@git.apache.org>.
Github user ctubbsii commented on the pull request:

    https://github.com/apache/accumulo/pull/43#issuecomment-219140206
  
    ACCUMULO-3970


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] accumulo pull request: Accumulo 3970

Posted by ctubbsii <gi...@git.apache.org>.
Github user ctubbsii commented on the pull request:

    https://github.com/apache/accumulo/pull/43#issuecomment-219140064
  
    Test comment


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---