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 <[email protected]> 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" <[email protected] > <javascript:_e(%7B%7D,'cvml','[email protected]');>> 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
