[
https://issues.apache.org/jira/browse/CASSANDRA-10852?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15056036#comment-15056036
]
Aleksey Yeschenko commented on CASSANDRA-10852:
-----------------------------------------------
Not a review, however:
1. You are not allowed to add new abstract interface methods to public APIs.
For 3.x, though, as we are forcing Java 8, you can use default methods.
2. {{@Override}} missing in the implementations once you switch to {{default}}
> Add requireAuthorization method to IAuthorizer
> ----------------------------------------------
>
> Key: CASSANDRA-10852
> URL: https://issues.apache.org/jira/browse/CASSANDRA-10852
> Project: Cassandra
> Issue Type: New Feature
> Reporter: Mike Adamson
> Assignee: Mike Adamson
> Priority: Minor
> Fix For: 3.2
>
>
> The {{IAuthenticator}} interface has a {{requireAuthentication}} to indicate
> whether an implementation requires an explicit login. For consistency it
> would be useful for 3rd party implementers if the {{IAuthorizer}} had a
> similar method {{requireAuthorization}} that would indicate if the
> implementation required explicit authorization.
> This would mean that we could remove and explicit {{instanceof}} checks for
> {{AllowAllAuthenticator}} and {{AllowAllAuthorizer}} in the code.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
