Return-Path: Delivered-To: apmail-geronimo-dev-archive@www.apache.org Received: (qmail 35667 invoked from network); 6 Jun 2007 15:41:18 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 6 Jun 2007 15:41:18 -0000 Received: (qmail 34014 invoked by uid 500); 6 Jun 2007 15:41:19 -0000 Delivered-To: apmail-geronimo-dev-archive@geronimo.apache.org Received: (qmail 33972 invoked by uid 500); 6 Jun 2007 15:41:19 -0000 Mailing-List: contact dev-help@geronimo.apache.org; run by ezmlm Precedence: bulk list-help: list-unsubscribe: List-Post: Reply-To: dev@geronimo.apache.org List-Id: Delivered-To: mailing list dev@geronimo.apache.org Received: (qmail 33961 invoked by uid 99); 6 Jun 2007 15:41:19 -0000 Received: from herse.apache.org (HELO herse.apache.org) (140.211.11.133) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 06 Jun 2007 08:41:19 -0700 X-ASF-Spam-Status: No, hits=0.0 required=10.0 tests= X-Spam-Check-By: apache.org Received-SPF: pass (herse.apache.org: local policy) Received: from [208.97.132.119] (HELO spaceymail-a1.g.dreamhost.com) (208.97.132.119) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 06 Jun 2007 08:41:14 -0700 Received: from [192.168.15.15] (cpe-76-167-174-30.socal.res.rr.com [76.167.174.30]) by spaceymail-a1.g.dreamhost.com (Postfix) with ESMTP id AFA7D81771 for ; Wed, 6 Jun 2007 08:40:53 -0700 (PDT) Mime-Version: 1.0 (Apple Message framework v752.3) In-Reply-To: <162838.44543.qm@web31711.mail.mud.yahoo.com> References: <162838.44543.qm@web31711.mail.mud.yahoo.com> Content-Type: text/plain; charset=US-ASCII; delsp=yes; format=flowed Message-Id: Content-Transfer-Encoding: 7bit From: Dain Sundstrom 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, 6 Jun 2007 08:40:23 -0700 To: dev@geronimo.apache.org X-Mailer: Apple Mail (2.752.3) X-Virus-Checked: Checked by ClamAV on apache.org 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 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 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/