DomGarguilo commented on code in PR #64:
URL: https://github.com/apache/accumulo-access/pull/64#discussion_r1497692856
##########
src/main/java/org/apache/accumulo/access/AccessEvaluator.java:
##########
@@ -88,12 +90,12 @@ static AccessEvaluator of(Authorizations authorizations) {
}
/**
- * Creates an AccessEvaluator from an Authorizer object
+ * Creates an AccessEvaluator from a Predicate<String>
Review Comment:
Added to the javadoc in 4e80d25 to try to address this.
##########
src/main/java/org/apache/accumulo/access/AccessEvaluator.java:
##########
@@ -159,13 +161,4 @@ static AccessEvaluator of(String... authorizations) {
return new
AccessEvaluatorImpl(AccessEvaluatorImpl.convert(authorizations));
}
- /**
- * An interface that is used to check if an authorization seen in an access
expression is
Review Comment:
Added to the javadoc in 4e80d25 to try to address this.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
