Thomas Mueller updated OAK-4817:
    Attachment: OAK-4817.patch


This introduces a new class QueryEngineSettingsMBeanImpl that wraps 
QueryEngineSettings. And QueryEngineSettings no longer extends 
AnnotatedStandardMBean (only QueryEngineSettingsMBeanImpl is). There are wrap() 
and unwrap() methods, to convert a QueryEngineSettings to a 
QueryEngineSettingsMBeanImpl, and back.

[~edivad] and [~chetanm], could you review the patch please?

> QueryEngineSettings without MBean
> ---------------------------------
>                 Key: OAK-4817
>                 URL: https://issues.apache.org/jira/browse/OAK-4817
>             Project: Jackrabbit Oak
>          Issue Type: Improvement
>          Components: query
>            Reporter: Thomas Mueller
>            Assignee: Thomas Mueller
>             Fix For: 1.6
>         Attachments: OAK-4817.patch
> QueryEngineSettings is currently an MBean, and constructing a new instance is 
> expensive. This is seem in some oak-core security components. The MBean 
> functionality should be decoupled.

This message was sent by Atlassian JIRA

Reply via email to