[ https://issues.apache.org/jira/browse/ACCUMULO-259?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13182679#comment-13182679 ]
Keith Turner commented on ACCUMULO-259: --------------------------------------- It may be better to throw an exception instead of returning false when something does not work. A user of the API can forget to check the boolean and nothing will happen. If you do not properly handle the exception, it will percolate up. > ZKAuthentictor needs to be split up > ----------------------------------- > > Key: ACCUMULO-259 > URL: https://issues.apache.org/jira/browse/ACCUMULO-259 > Project: Accumulo > Issue Type: Improvement > Components: master, tserver > Reporter: John Vines > Assignee: Eric Newton > > Currently the ZKAuthenticator does both the checking of credentials as well > as the functionality of the Authenticator. We need to split those into two > different classes for improved testing as well as pluggability of the > security scheme. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira