Improving Node.js Loader Performance

sentry.engineering

23 pointsamalinovic5 comments

If I got it right, naming all our files with the extension `.mjs` solves the issue as well, right ?

Man I was expecting some massive result, where some plucky engineer unraveled a huge inefficiency buried all this time in node.

After all this work we get a 5% improvement??? With the addition of a new Json parsing library on top? Bruh

Too snarky to be insightful. Can you solve it better

No, I really think the author himself was thinking he would get a lot more out of it, otherwise he would've have buried the result all the way at the end of a giant wall of text.

In a way we all solved it better by not doing anything at all. Which doesn't introduce the Json library he added in. Which was probably why the pr was rejected

Also my nips are wet

5% is actually pretty big on the scale of Node

Reply on news.ycombinator.com