[jira] [Commented] (CASSANDRA-8722) Auth MBean needs to be registered

["Aleksey Yeschenko (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/CASSANDRA-8722?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14328573#comment-14328573 ] 

Aleksey Yeschenko commented on CASSANDRA-8722:
----------------------------------------------

Can't pass a class to registerMBean, so this doesn't work :\

I suggest you to rename AuthMBean to PermissionsCacheMBean, and have PermissionsCache implement it, then register in the constructror (it's a singleton, so it's fine).

Also, some nits:
- redundant 'this' in updateCache()
- should add {get,set}PermissionsUpdateInterval() to the mbean, for symmetry, too


> Auth MBean needs to be registered
> ---------------------------------
>
>                 Key: CASSANDRA-8722
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-8722
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Core
>            Reporter: Brandon Williams
>            Assignee: Brandon Williams
>            Priority: Minor
>             Fix For: 2.1.4
>
>         Attachments: 8722.txt
>
>
> In CASSANDRA-7968 we created this bean but forgot to register it :(  This also makes CASSANDRA-7977 unusable.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)