geronimo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Dain Sundstrom <d...@iq80.com>
Subject Re: mx4j logging init
Date Wed, 28 Feb 2007 18:17:06 GMT
Last I checked, the MX4j code wasn't compliant with the Java5 specs,  
but I don't think we were using it either.... IIRC you have to do  
something special to replace the JMX implementation in the vm.

-dain

On Feb 27, 2007, at 4:21 PM, Jason Dillon wrote:

> Okay, if we don't need it then we can save ~600k wooopie... and can  
> prolly remove a few hackish bits (like the logging init stuff).
>
> --jason
>
>
> On Feb 27, 2007, at 3:50 PM, Dain Sundstrom wrote:
>
>> With the exception of the remote deployer, we don't rely on JMX.   
>> The remote deployer uses JMX remoting to connect to the Geronimo  
>> deployment service.  Assuming that works, we should be ok using  
>> what the vm ships.
>>
>> -dain
>>
>> On Feb 27, 2007, at 1:27 PM, Jason Dillon wrote:
>>
>>> Good point... we should start ripping that stuff out, probably  
>>> save a wee bit of space in the assembly.
>>>
>>> Though... by doing so we are at the mercy of the JVM to provide a  
>>> JMX system that doesn't suck ass (horrible bugs, etc).
>>>
>>> --jason
>>>
>>>
>>> On Feb 27, 2007, at 12:51 AM, Dain Sundstrom wrote:
>>>
>>>> We don't need mx4j in 2.0 since jmx is in the vm starting with  
>>>> Java5.
>>>>
>>>> -dain
>>>>
>>>> On Feb 26, 2007, at 7:02 PM, Jason Dillon wrote:
>>>>
>>>>> Anyone know if we can replace these bits in  
>>>>> GeronimoLogging.initialize()
>>>>>
>>>>> <snip>
>>>>> try {
>>>>>     Class clazz = Class.forName("mx4j.log.Log");
>>>>>     Class paramClazz = Class.forName("mx4j.log.Logger");
>>>>>     Method method = clazz.getDeclaredMethod("redirectTo", new  
>>>>> Class[] {paramClazz});
>>>>>     paramClazz = Class.forName("mx4j.log.CommonsLogger");
>>>>>     method.invoke(null, new Object[] {paramClazz.newInstance()});
>>>>> } catch (ClassNotFoundException e) {
>>>>>     // MX4J is not present.
>>>>> } catch (Exception e) {
>>>>>     throw (AssertionError) new AssertionError("Cannot force  
>>>>> MX4J to use commons logging.").initCause(e);
>>>>> }
>>>>> </snip>
>>>>>
>>>>> With:
>>>>>
>>>>> <snip>
>>>>> System.setProperty("mx4j.log.prototype",  
>>>>> "mx4j.log.CommonsLogger");
>>>>> </snip>
>>>>>
>>>>> --jason
>>>>
>>>
>>
>


Mime
View raw message