Skip to content

Modified VisibilityFilter to handle multiple Authorizations objects#6402

Merged
dlmarion merged 8 commits into
apache:mainfrom
dlmarion:multi-auth-vis-filter
Jul 20, 2026
Merged

Modified VisibilityFilter to handle multiple Authorizations objects#6402
dlmarion merged 8 commits into
apache:mainfrom
dlmarion:multi-auth-vis-filter

Conversation

@dlmarion

@dlmarion dlmarion commented May 29, 2026

Copy link
Copy Markdown
Contributor

Accumulo Access provides an AccessEvaluator
that accepts a Collection of Sets of Strings (effectively
Collection<Authorizations>). Modified the existing user
VisibilityFilter to accept a set of Authorizations and
use the new AccessEvaluator.

Accumulo Access provides an AccessEvaluator that accepts
a Collection of Sets of Strings (effectively
Collection<Authorizations>). The existing user
VisibilityFilter only accepts a single Authorizations
object. This new visibility filter accepts a set of
Authorizations in a single filter.
@dlmarion dlmarion added this to the 4.0.0 milestone May 29, 2026
@dlmarion
dlmarion requested review from ctubbsii and keith-turner May 29, 2026 17:03
@dlmarion dlmarion self-assigned this May 29, 2026
@dlmarion dlmarion changed the title Added MultiAuthVisibilityFilter to user iterator package Modified VisibilityFilter to handle multiple Authorizations objects Jun 8, 2026
Comment thread core/src/main/java/org/apache/accumulo/core/iterators/user/VisibilityFilter.java Outdated

@ctubbsii ctubbsii left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The concatenated set of serialized Authorizations looks good to me. I think the description needs work, though.

Comment thread core/src/main/java/org/apache/accumulo/core/iterators/user/VisibilityFilter.java Outdated
Comment thread core/src/main/java/org/apache/accumulo/core/iterators/user/VisibilityFilter.java Outdated
@dlmarion
dlmarion merged commit 64995a6 into apache:main Jul 20, 2026
8 checks passed
@dlmarion
dlmarion deleted the multi-auth-vis-filter branch July 20, 2026 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants