geronimo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From David Blevins <david.blev...@visi.com>
Subject Re: [jira] Created: (GERONIMO-3445) Verify log levels can be changed for openejb related log categories
Date Thu, 06 Sep 2007 21:47:44 GMT
Cool.  I've updated the Geronimo side as follows:

Index: src/main/java/org/apache/geronimo/openejb/OpenEjbSystemGBean.java
===================================================================
--- src/main/java/org/apache/geronimo/openejb/ 
OpenEjbSystemGBean.java   (revision 570242)
+++ src/main/java/org/apache/geronimo/openejb/ 
OpenEjbSystemGBean.java   (working copy)
@@ -104,7 +104,8 @@
          }
          System.setProperty("duct tape","");
          System.setProperty("admin.disabled", "true");
-
+        System.setProperty("openejb.logger.external", "true");
+
          setDefaultProperty("openejb.deploymentId.format",  
"{moduleId}/{ejbName}");
          setDefaultProperty("openejb.jndiname.strategy.class",  
"org.apache.openejb.assembler.classic.JndiBuilder$TemplatedStrategy");
          setDefaultProperty("openejb.jndiname.format",  
"{deploymentId}/{interfaceClass}");


On Sep 6, 2007, at 1:52 PM, Karan Malhi wrote:

> Yes it should work.
>
> On 9/6/07, David Blevins <david.blevins@visi.com> wrote:
>>
>> On Sep 6, 2007, at 9:04 AM, Karan Malhi wrote:
>>
>>> This is because Geronimo also uses the same classes as we use in
>>> embedded mode. So it will not be possible to set the SystemProperty
>>> there. For example, OpenEjbSystemGBean also uses SystemInstance, but
>>> our code also uses SystemInstance heavily. So setting it in
>>> SystemInstance will not lead to the desired behaviour.
>>>
>>> We can however try the following options:
>>>
>>> 1. Ask Geronimo team to set this property in
>>> org.apache.geronimo.openejb.OpenEjbSystemGBean
>>
>> Not sure I followed the explanation above.  Ideally we'd be able to
>> use option 1 here as that's what we do for other properties.
>> Setting openejb.logging.external=true as a plain system property in
>> OpenEjbSystemGBean should work right?
>>
>> -David
>>
>>
>>
>
>
> -- 
> Karan Singh Malhi
>


Mime
View raw message