jsonp icon indicating copy to clipboard operation
jsonp copied to clipboard

Cache JsonProvider.provider() result as per javadoc recommendation

Open gbloisi opened this issue 7 years ago • 3 comments

I got through this because I found a performance hostpot in calling Json methods (about 500 microsec per call on a i5)

gbloisi avatar Mar 18 '18 00:03 gbloisi

+1, I'm serialising a large object and it takes about 2000ms on my machine currently. Switching to this implementation drops it down to 50ms.

rogersnm avatar Apr 06 '18 15:04 rogersnm

See https://github.com/eclipse-ee4j/jsonp/pull/80 it seems there already is a similar PR in Jakarta EE. I doubt that this repository results in any new updates or releases.

keilw avatar Apr 06 '18 15:04 keilw

@lukasj @bravehorsie Recently there were some similar PRs in different projects. I am not 100% convinced that it's needed. Folks, I want you to review it before letting it go.

m0mus avatar Apr 06 '18 15:04 m0mus