keith-turner commented on code in PR #64:
URL: https://github.com/apache/accumulo-access/pull/64#discussion_r1496546145


##########
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:
   This javadoc should be moved (with rewording) to the method that now takes a 
predicate.  Need to explain what the purpose of the predicate is, which this 
javadoc kinda does.



-- 
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]

Reply via email to