[
https://issues.apache.org/jira/browse/IGNITE-8776?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16535043#comment-16535043
]
kcheng.mvp edited comment on IGNITE-8776 at 7/6/18 4:00 PM:
------------------------------------------------------------
[~slukyanov]
Everything is green
https://ci.ignite.apache.org/viewLog.html?buildId=1462771&tab=buildResultsDiv&buildTypeId=IgniteTests24Java8_RunBasicTests
Can you help me do a code review?
was (Author: kcheng.mvp):
[~slukyanov]
Everything is greep
https://ci.ignite.apache.org/viewLog.html?buildId=1462771&tab=buildResultsDiv&buildTypeId=IgniteTests24Java8_RunBasicTests
Can you help me do a code review?
> Eviction policy MBeans are never registered if evictionPolicyFactory is used
> ----------------------------------------------------------------------------
>
> Key: IGNITE-8776
> URL: https://issues.apache.org/jira/browse/IGNITE-8776
> Project: Ignite
> Issue Type: Bug
> Affects Versions: 2.5
> Reporter: Stanislav Lukyanov
> Assignee: kcheng.mvp
> Priority: Minor
> Labels: newbie
>
> Eviction policy MBeans, such as LruEvictionPolicyMBean, are never registered
> if evictionPolicyFactory is set instead of evictionPolicy (the latter is
> deprecated).
> This happens because GridCacheProcessor::registerMbean attempts to find
> either an *MBean interface or IgniteMBeanAware interface on the passed
> object. It works for LruEvictionPolicy but not for LruEvictionPolicyFactory
> (which doesn't implement these interfaces).
> The code needs to be adjusted to handle factories correctly.
> New tests are needed to make sure that all standard beans are registered
> (IgniteKernalMbeansTest does that for kernal mbeans - need the same for cache
> beans).
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
