search

reactiverse / es4x

🚀 fast JavaScript 4 Eclipse Vert.x
https://reactiverse.io/es4x/
Apache License 2.0
878 stars 75 forks source link

Remove JSON polyfill #520

Closed pmlopes closed 3 years ago

pmlopes commented 3 years ago

The JSON polyfill allows us to encode json directly to a Vert.x Buffer, which would reduce the number of copies and generated garbage. It looks like this polyfill introduces a de-optimization on the native code, which ends up slowing down the serialization.

We need to investigate if the need for this encode to buffer / parse from buffer is worth in terms of performance versus not having it at all.

pmlopes commented 3 years ago

Fixed by: 3e4a7aec31354ae6ec2cfa77fd1aabeaca4d70b3