|
| 1 | +# Node.js Performance Team Meeting 2023-04-03 |
| 2 | + |
| 3 | +## Links |
| 4 | + |
| 5 | +* **Recording**: https://www.youtube.com/watch?v=MVqHVCv-rnQ |
| 6 | +* **GitHub Issue**: https://github.com/nodejs/performance/issues/68 |
| 7 | + |
| 8 | +## Present |
| 9 | + |
| 10 | +* Performance team: @nodejs/performance |
| 11 | +* Yagiz Nizipli: @anonrig |
| 12 | +* Stephen Belanger: @qard |
| 13 | +* Robert Nagy: @ronag |
| 14 | +* Debadree Chatterjee @debadree25 |
| 15 | + |
| 16 | + |
| 17 | +## Agenda |
| 18 | + |
| 19 | +## Announcements |
| 20 | + |
| 21 | +* Ada v2 is released. Announcement post is available at http://yagiz.co/announcing-ada-url-parser-v2-0/ |
| 22 | +* V8 11.3 brings `String.prototype.toWellFormed()` support with 70-80% faster `toUSVString` |
| 23 | +* Increasing the default watermark size for version 20. It provides a performance boost! |
| 24 | + |
| 25 | +*Extracted from **performance-agenda** labelled issues and pull requests from the **nodejs org** prior to the meeting. |
| 26 | + |
| 27 | +### nodejs/performance |
| 28 | + |
| 29 | +* add/remove handlers are slow on EventTarget [#60](https://github.com/nodejs/performance/issues/60) |
| 30 | + * It would be great to add a fast path for AbortSignal to have only one `abort` listener. |
| 31 | + * There is no one working on it right now. |
| 32 | + * Debadree attempted on adding a hashmap based approach but did not help with any performance impacts. |
| 33 | + * It would be great to have a benchmark for adding and removing a single event listener (the current benchmarks adds multiple listeners and then dispatches), because the proposed benchmark would never traverse the linkedlist. |
| 34 | +* Session proposals for the next Collaborator Summit - what should we submit [#64](https://github.com/nodejs/performance/issues/64) |
| 35 | +* Bun & Deno Comparison [#55](https://github.com/nodejs/performance/issues/55) |
| 36 | + * Yagiz proposed adding his own performance benchmarks to the performance repository. Will keep this for visibility. |
| 37 | +* AbortSignal pattern is slow [#44](https://github.com/nodejs/performance/issues/44) |
| 38 | + * Closing it since it is a duplicate of add/remove handlers for EventTarget |
| 39 | +* Module resolution [#39](https://github.com/nodejs/performance/issues/39) |
| 40 | + * Put this on hold until off-thread loader pull request is landed. |
| 41 | +* EventTarget [#32](https://github.com/nodejs/performance/issues/32) |
| 42 | + * Closing it due to Santiago’s amazing contributions. |
| 43 | +* fetch [#11](https://github.com/nodejs/performance/issues/11) |
| 44 | + * Robert: I don’t know where to start optimizing. There’s more work to be done. Maybe we can ask Rafael on making a new analysis on undici fetch? |
| 45 | + |
| 46 | + |
| 47 | +## Q&A, Other |
| 48 | + |
| 49 | +## Upcoming Meetings |
| 50 | + |
| 51 | +* **Node.js Foundation Calendar**: https://nodejs.org/calendar |
| 52 | + |
| 53 | +Click `+GoogleCalendar` at the bottom right to add to your own Google calendar. |
| 54 | + |
0 commit comments