jakartaee / jsonp-api

Jakarta JSON Processing
https://eclipse.org/ee4j/jsonp
Other
141 stars 61 forks source link

Issue #205 - improve performance by allowing client to set the JsonProvider instance used by javax.json.Json #206

Closed JohnTimm closed 4 years ago

keilw commented 5 years ago

On a bird's eye view it is not so different from how JSR 385 and others cache providers. I'd like someone from Oracle to check it out if possible.

JohnTimm commented 4 years ago

@alexsm82 this hasn't moved at all and now it appears to have conflicts with master. where do we go from here?

alexsm82 commented 4 years ago

@m0mus any feedback on this approach?

m0mus commented 4 years ago

Sorry for delay. It looks fine to me, but I want @lukasj to review it. Also, please add tests.

JohnTimm commented 4 years ago

Closing this PR in favor of #218.