geronimo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Clebert Suconic <clebert.suco...@gmail.com>
Subject Re: PR Merge?
Date Fri, 19 Aug 2016 13:40:37 GMT
The patch is caching it with a soft cache. Meaning the impl will go away
when memory is needed and a class loader gone.


The performance of having to load the class on every operation is really
heavy.

On Friday, August 19, 2016, Romain Manni-Bucau <rmannibucau@gmail.com>
wrote:

> Hi John
>
> I implement it this way to work in all cases and deployment without
> leaking an impl (this patch does)
>
> The fix is in artemis I suspect: Json.write/read shouldnt be used bit they
> should go through the factory.
>
> Hope it helps.
>
> Le 19 août 2016 04:11, "John D. Ament" <johndament@apache.org
> <javascript:_e(%7B%7D,'cvml','johndament@apache.org');>> a écrit :
>
>> Hi,
>>
>> I was wondering, would anyone be able to review this PR and perhaps merge
>> it?  After I ported Artemis to use Johnzon, Clebert noticed some
>> performance issues where the service was being looked up always.  His PR is
>> to help resolve that.
>>
>> https://github.com/apache/geronimo-specs/pull/4
>>
>> John
>>
>

-- 
Clebert Suconic

Mime
View raw message