Экспорт лениво инициализированного бина (который реализует SelfNaming и аннотируется с помощью аннотации ManagedResource) дает IllegalStateException - PullRequest
Новая
       80

Экспорт лениво инициализированного бина (который реализует SelfNaming и аннотируется с помощью аннотации ManagedResource) дает IllegalStateException

0 голосов
/ 10 декабря 2018

У меня есть bean-компонент,

  • с аннотацией ManagedResource
  • , с ленивой инициализацией
  • , SelfNaming

Я экспортирую его, используя AnnotationMBeanExporter.

. Все это хорошо работает, когда я использую версию 4.3.16.RELEASE, но когда я обновил свою весеннюю версию до 5.0.5.RELEASE или 5.1.3.RELEASE,код начал давать мне IllegalStateException.

Мое определение компонента и контекстный источник Spring.xml выглядит следующим образом:

SampleBean.java: 

package com.jmx.trial.dummybeans;

import org.springframework.jmx.export.annotation.ManagedResource;
import org.springframework.jmx.export.naming.SelfNaming;

import javax.management.MalformedObjectNameException;
import javax.management.ObjectName;

@ManagedResource
public class SampleBean implements SelfNaming {
    @Override
    public ObjectName getObjectName() throws MalformedObjectNameException {
        return new ObjectName("com.jmx.trial:name=sampleBean");
    }
}

application-context.xml 

<?xml version="1.0" encoding="UTF-8"?>
<beans xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
       xmlns="http://www.springframework.org/schema/beans"
       xsi:schemaLocation="http://www.springframework.org/schema/beans
        http://www.springframework.org/schema/beans/spring-beans.xsd">

    <bean id="server" class="org.springframework.jmx.support.MBeanServerFactoryBean"/>

    <bean id="exporter" class="org.springframework.jmx.export.annotation.AnnotationMBeanExporter">
        <property name="server" ref="server"/>
    </bean>

    <bean id="sampleBean" class="com.jmx.trial.dummybeans.SampleBean" lazy-init="true"/>

</beans>

Я понимаю, что добавлена ​​дополнительная проверка здесь , и это вызывает IllegalStateException, но я не совсем уверен, почему это было добавлено.

Трассировка стека выглядит следующим образом: 

org.springframework.jmx.export.UnableToRegisterMBeanException: Unable to register MBean [sampleBean] with key 'sampleBean'; nested exception is java.lang.IllegalStateException: Not initialized

    at org.springframework.jmx.export.MBeanExporter.registerBeanNameOrInstance(MBeanExporter.java:625)
    at org.springframework.jmx.export.MBeanExporter.lambda$registerBeans$2(MBeanExporter.java:551)
    at java.base/java.util.HashMap.forEach(HashMap.java:1336)
    at org.springframework.jmx.export.MBeanExporter.registerBeans(MBeanExporter.java:551)
    at org.springframework.jmx.export.MBeanExporter.afterSingletonsInstantiated(MBeanExporter.java:434)
    at org.springframework.beans.factory.support.DefaultListableBeanFactory.preInstantiateSingletons(DefaultListableBeanFactory.java:863)
    at org.springframework.context.support.AbstractApplicationContext.finishBeanFactoryInitialization(AbstractApplicationContext.java:863)
    at org.springframework.context.support.AbstractApplicationContext.refresh(AbstractApplicationContext.java:546)
    at org.springframework.context.support.ClassPathXmlApplicationContext.<init>(ClassPathXmlApplicationContext.java:144)
    at org.springframework.context.support.ClassPathXmlApplicationContext.<init>(ClassPathXmlApplicationContext.java:85)
    at com.jmx.trial.MBeanExporterTest.testBeanExportedWithXml(MBeanExporterTest.java:79)
    at com.jmx.trial.MBeanExporterTest.testForLazyAutoDetectWithSelfNaming(MBeanExporterTest.java:44)
    at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
    at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
    at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
    at java.base/java.lang.reflect.Method.invoke(Method.java:566)
    at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
    at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
    at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
    at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
    at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:325)
    at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:78)
    at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:57)
    at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
    at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
    at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
    at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
    at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
    at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
    at org.junit.runner.JUnitCore.run(JUnitCore.java:137)
    at com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:68)
    at com.intellij.rt.execution.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:47)
    at com.intellij.rt.execution.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:242)
    at com.intellij.rt.execution.junit.JUnitStarter.main(JUnitStarter.java:70)
Caused by: java.lang.IllegalStateException: Not initialized
    at org.springframework.util.Assert.state(Assert.java:73)
    at org.springframework.jmx.export.MBeanExporter$NotificationPublisherAwareLazyTargetSource.postProcessTargetObject(MBeanExporter.java:1115)
    at org.springframework.aop.target.LazyInitTargetSource.getTarget(LazyInitTargetSource.java:72)
    at org.springframework.jmx.export.MBeanExporter$NotificationPublisherAwareLazyTargetSource.getTarget(MBeanExporter.java:1103)
    at org.springframework.aop.framework.CglibAopProxy$DynamicUnadvisedInterceptor.intercept(CglibAopProxy.java:475)
    at com.jmx.trial.dummybeans.SampleBean$$EnhancerBySpringCGLIB$$9cd1c95b.getObjectName(<generated>)
    at org.springframework.jmx.export.MBeanExporter.getObjectName(MBeanExporter.java:752)
    at org.springframework.jmx.export.MBeanExporter.registerLazyInit(MBeanExporter.java:726)
    at org.springframework.jmx.export.MBeanExporter.registerBeanNameOrInstance(MBeanExporter.java:596)
    ... 33 more

Мои эксперименты:

  • Когда я не лениво инициализирую sampleBean, я не получаю это IllegalStateException.
  • Когда я аннотирую SampleBean с ManagedResource(objectName = "com.jmx.trial:name=sampleBean") и заставляю SampleBean не реализовать SelfNaming интерфейс, снова я не получаю IllegalStateException

Хотя нетКонечно, но я думаю, что реализация интерфейса SelfNaming не очень хорошая идея, так как javadocs интерфейса SelfNaming говорит, что:

Этот интерфейс в основном предназначен для внутреннего использования.

Я не уверен, если это проблема с использованием интерфейса SelfNaming, или я делаю что-то в корне неправильно.Не могли бы вы объяснить это поведение и указать мне на основные принципы, которые я упускаю.

PS: некоторые из моих выводов: (могут быть не связаны) С здесь Я нашел этот пункт предостережения:

Не используйте прокси-серверы AOP на основе интерфейса в сочетании с автоматическим определением аннотаций JMX в ваших классах компонентов.

Я не совсем понимаю, ноПравило, что я нарушаю?

1 Ответ

0 голосов
/ 10 декабря 2018

Возможно, это ошибка;injectNotificationPublisherIfNecessary предпринимает какие-либо действия только в том случае, если ваш класс также реализует NotificationPublisherAware.

Это было бы недопустимо, поскольку у нас пока нет objectName, поскольку бин ленив и это нарушит контрактдля ModelMBeanNotificationPublisher.

Я не знаю, может ли postProcessTargetObject проигнорировать утверждение, если ресурс не реализует интерфейс, осведомленный об издателе, или нужно ли выполнять инъекцию издателя также лениво.Я недостаточно хорошо знаю внутренности.

Я предлагаю вам открыть вопрос SPR JIRA , чтобы Spring Team могла взглянуть и дать вам окончательный ответ.

...