ctubbsii opened a new issue, #59: URL: https://github.com/apache/accumulo-access/issues/59
`IllegalAccessExpressionException` starts with `IllegalAccess`, which has a particular meaning, and I think the current name could cause confusion. It could be renamed, so as to make it clear that it's an `AccessExpression` exception and not an `IllegalAccess` exception. Also, the supertype is `PatternSyntaxException`, which is a type that comes from the `java.util.regex` package, and specifically pertains to exceptions when dealing with regular expressions. I understand we're using it to help display helpful error messages that show where the problem occurred (if we can), but while that's fine in an internal API to construct a log message, it's probably too confusing when used in a public API for users to handle. -- 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]
