You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@geronimo.apache.org by Jacek Laskowski <ja...@laskowski.net.pl> on 2007/01/19 08:17:28 UTC

Re: svn commit: r497717 - in /geronimo/server/trunk: ./ configs/openejb/src/plan/ modules/geronimo-openejb/ modules/geronimo-openejb/src/main/java/org/apache/geronimo/openejb/ modules/geronimo-openejb/src/main/resources/META-INF/

On 1/19/07, dain@apache.org <da...@apache.org> wrote:
> Author: dain
> Date: Thu Jan 18 22:51:43 2007
> New Revision: 497717
>
> URL: http://svn.apache.org/viewvc?view=rev&rev=497717
> Log:
> Stubbed out ejb daemon gbean
...
> +public class EjbDaemonGBean implements NetworkConnector, GBeanLifecycle {
...
> +    public void doStart() throws Exception {
> +        Properties properties = new Properties();
> +        properties.setProperty("server", "org.apache.openejb.server.ejbd.EjbServer");

I wish we didn't include any unmodifiable values of GBean properties
like the above. Could it be overriden via the gbean's server
attribute?

Jacek

-- 
Jacek Laskowski
http://www.JacekLaskowski.pl

Re: svn commit: r497717 - in /geronimo/server/trunk: ./ configs/openejb/src/plan/ modules/geronimo-openejb/ modules/geronimo-openejb/src/main/java/org/apache/geronimo/openejb/ modules/geronimo-openejb/src/main/resources/META-INF/

Posted by David Blevins <da...@visi.com>.
On Jan 18, 2007, at 11:17 PM, Jacek Laskowski wrote:

> On 1/19/07, dain@apache.org <da...@apache.org> wrote:
>> Author: dain
>> Date: Thu Jan 18 22:51:43 2007
>> New Revision: 497717
>>
>> URL: http://svn.apache.org/viewvc?view=rev&rev=497717
>> Log:
>> Stubbed out ejb daemon gbean
> ...
>> +public class EjbDaemonGBean implements NetworkConnector,  
>> GBeanLifecycle {
> ...
>> +    public void doStart() throws Exception {
>> +        Properties properties = new Properties();
>> +        properties.setProperty("server",  
>> "org.apache.openejb.server.ejbd.EjbServer");
>
> I wish we didn't include any unmodifiable values of GBean properties
> like the above. Could it be overriden via the gbean's server
> attribute?

That code didn't actually work anyway, so I yanked it.  Check out the  
new gbean.

-David