geronimo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Dain Sundstrom <d...@iq80.com>
Subject Re: GBeanInfoBuilder [was Re: svn commit: r499201 - in /geronimo/server/trunk/modules/geronimo-kernel/src: main/java/org/apache/geronimo/gbean/ main/java/org/apache/geronimo/gbean/runtime/ test/java/org/apache/geronimo/gbean/
Date Wed, 06 Jun 2007 15:40:23 GMT
Sounds good to me.  BTW, if you don't want the get/set methods to  
show up in a JMX console, just don't add them to the  MBeanInfo.

-dain

On Jun 5, 2007, at 5:23 PM, Anita Kulshreshtha wrote:

> Dain,
>    I was able to observe this problem on webconsole. I would like to
> reapply this patch with following modifications:
> 1. Leave addOperation(String ..) ASIS, i.e., getters/setters must be
> added as operation explicitly to be able to invoke them as operation.
> 2. Fix tests to demonstrate this behavior.
>    Any objections?
>
> Thanks
> Anita
>
>
> --- anita kulshreshtha <a_kulshre@yahoo.com> wrote:
>
>>    We are not doing this at present. Please see:
>>
> http://www.nabble.com/Re%3A-svn-commit%3A-r499201---in--geronimo- 
> server-trunk-modules-geronimo-kernel-src%3A-main-java-org-apache- 
> geronimo-gbean--main-java-org-apache-geronimo-gbean-runtime--test- 
> java-org-apache-geronimo-gbean--p8769582.html
>>    The above output is for rev 502653 without my patch. The
>> getValue()
>> method is a valid getter.
>>
>> Thanks
>> Anita
>>
>> --- Dain Sundstrom <dain@iq80.com> wrote:
>>
>>> All methods regardless of if they are valid getters or setters
>> should
>>>
>>> be exposed via the invoke method (using an operation info).
>>>
>>> -dain
>
>
>
>
> ______________________________________________________________________ 
> ______________
> Looking for a deal? Find great prices on flights and hotels with  
> Yahoo! FareChase.
> http://farechase.yahoo.com/


Mime
View raw message