[ 
https://issues.apache.org/jira/browse/OAK-3519?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

angela updated OAK-3519:
------------------------
    Attachment: OAK-3519.patch

[~tripod], since you are the original author of the SyncMBeanImpl it would be 
good if you take a look at the proposed patch. thanks

> Some improvement to SyncMBeanImpl
> ---------------------------------
>
>                 Key: OAK-3519
>                 URL: https://issues.apache.org/jira/browse/OAK-3519
>             Project: Jackrabbit Oak
>          Issue Type: Improvement
>          Components: auth-external
>            Reporter: angela
>            Assignee: angela
>            Priority: Minor
>         Attachments: OAK-3519.patch
>
>
> IMHO the {{SyncMBeanImpl}} deserves some improvement:
> - code duplication
> - overall readability
> - removing catching {{Exception}}
> - throwing {{IllegalArgument}} when no argument has been provided instead of 
> a dedicated {{RuntimeException}} indicating that the operation could not be 
> successfully completed.



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

Reply via email to