Closed johnsonr closed 7 years ago
Radical performance improvement when working with graph nodes.
This reduces the new benchmark in LinkedJsonGraphDeserializerTest from 10,000ms to around 60.
LinkedJsonGraphDeserializerTest
The problem was in inadvertently calling into ServiceLoader via jsSafeCommittingProxy -> Typed.typeFor -> TypeRegistry.findByName(t)
ServiceLoader
jsSafeCommittingProxy
Typed.typeFor
TypeRegistry.findByName(t)
Note: See @cdupuis comment on ServiceLoaderBackedExtensionProvider. Does the new behavior affect this?
ServiceLoaderBackedExtensionProvider
Radical performance improvement when working with graph nodes.
This reduces the new benchmark in
LinkedJsonGraphDeserializerTestfrom 10,000ms to around 60.The problem was in inadvertently calling into
ServiceLoaderviajsSafeCommittingProxy->Typed.typeFor->TypeRegistry.findByName(t)Note: See @cdupuis comment on
ServiceLoaderBackedExtensionProvider. Does the new behavior affect this?