[infinispan-dev] ResourceDMBean.invoke() should only allow for methods marked with @ManagedOperation to be called from MBeanServer.invoke()

[Manik Surtani]

I agree.  Being close to the spec is important.

On 9 Sep 2009, at 15:03, Galder Zamarreno wrote:

>
>
> On 09/09/2009 10:57 AM, Mircea Markus wrote:
>>
>> On Sep 8, 2009, at 8:38 PM, Vladimir Blagojevic wrote:
>>
>>> On 09-09-08 12:58 PM, Galder Zamarreno wrote:
>>>> Yeah, this is correct. In a standard MBean, the operation called
>>>> must be
>>>> part of the MBean interface, otherwise it fails. So, it must be
>>>> exposed
>>>> somehow. In a required model mbean, the operation must have been
>>>> defined
>>>> as an MBean method.
>>>>
>>>> So, yeah, ResourceDMBean.invoke() needs to check whether the method
>>>> called is a managed operation or not. I'll create a jira for this
>>>> asap.
>>>>
>>> Bela and I deviated from spec intentionally because we liked the  
>>> fact
>>> that you could expose all class methods by simply annotating class
>>> with
>>> @MBean. Either way it should be simple to fix.
>> yep, perhaps we should have an exposeByDefault attribute on @MBean
>> that is defaulted to false.
>
> Hmmm, don't think so. We shouldn't try to use JMX as a swiss knife. If
> we wanna expose something, we should annotate it accordingly,  
> otherwise
> it shouldn't be called via JMX.
>
>>>
>>> Cheers
>>> _______________________________________________
>>> infinispan-dev mailing list
>>> infinispan-dev at lists.jboss.org
>>> https://lists.jboss.org/mailman/listinfo/infinispan-dev
>>
>> _______________________________________________
>> infinispan-dev mailing list
>> infinispan-dev at lists.jboss.org
>> https://lists.jboss.org/mailman/listinfo/infinispan-dev
>
> -- 
> Galder Zamarreño
> Sr. Software Engineer
> Infinispan, JBoss Cache
> _______________________________________________
> infinispan-dev mailing list
> infinispan-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/infinispan-dev

--
Manik Surtani
manik at jboss.org
Lead, Infinispan
Lead, JBoss Cache
http://www.infinispan.org
http://www.jbosscache.org