|
| 1 | +# Node.js Performance Team Meeting 2023-10-16 |
| 2 | + |
| 3 | +## Links |
| 4 | + |
| 5 | +* **Recording**: https://www.youtube.com/watch?v=QRaFTe-CLeU |
| 6 | +* **GitHub Issue**: https://github.com/nodejs/performance/issues/124 |
| 7 | + |
| 8 | +## Present |
| 9 | + |
| 10 | +* Performance team: @nodejs/performance |
| 11 | +* Yagiz Nizipli @anonrig |
| 12 | +* Debadree Chatterjee @debadree25 |
| 13 | +* Aras Abbasi @uzlopak |
| 14 | +* Rafael Gonzaga @RafaelGSS |
| 15 | +* André Alves @andremralves |
| 16 | + |
| 17 | +## Agenda |
| 18 | + |
| 19 | +## Announcements |
| 20 | + |
| 21 | +*Extracted from **performance-agenda** labelled issues and pull requests from the **nodejs org** prior to the meeting. |
| 22 | + |
| 23 | +### nodejs/performance |
| 24 | + |
| 25 | +* Bring back clonefile on Mac OS X [#110](https://github.com/nodejs/performance/issues/110) |
| 26 | + * Removed the label and tagged libuv team for a follow-up of the issue. |
| 27 | +* Performance of `node:fs` [#106](https://github.com/nodejs/performance/issues/106) |
| 28 | + * There are still some sync methods not optimized for error handling. Pull requests are welcome. |
| 29 | + * There are a lot of optimization opportunities in fs.promises api that involves reducing the C++ to JS bridge communication. |
| 30 | +* Fast path for `--no-deprecation` [#90](https://github.com/nodejs/performance/issues/90) |
| 31 | + * @aduh95 said “I’m not sure it’s relevant to spend any time optimizing deprecated features”. Yagiz & Rafael agrees. |
| 32 | + * Removing from the agenda and closing upon consensus. |
| 33 | +* Performance of WHATWG ReadableStream.read() [#82](https://github.com/nodejs/performance/issues/82) |
| 34 | + * Debadree worked on the issue, but couldn’t find an optimization without any breaking changes. |
| 35 | +* MakeCallback is very slow [#24](https://github.com/nodejs/performance/issues/24) |
| 36 | + * No new updates. Removing it from the agenda. |
| 37 | + |
| 38 | +## Q&A, Other |
| 39 | + |
| 40 | +* @uzlopak mentioned his improving hideStackFrames pull request. Asked for some feedback around the solution and/or document the existing solution. Ref: https://github.com/nodejs/node/pull/49990 |
| 41 | + * Rafael recommended creating a benchmark using async_hooks since microbenchmarks might not show the real truth about it. |
| 42 | +* Yagiz needs some help on Windows in order to unblock a PR opened 3 weeks ago. Ref: https://github.com/nodejs/node/pull/49962 |
| 43 | +* Rafael mentioned distributed benchmarking CI issue on Build repository https://github.com/nodejs/build/issues/3483 |
| 44 | + * Yagiz: fs benchmarks take 10+ hours - https://github.com/nodejs/node/issues/49689 |
| 45 | + |
| 46 | +## Upcoming Meetings |
| 47 | + |
| 48 | +* **Node.js Foundation Calendar**: https://nodejs.org/calendar |
| 49 | + |
| 50 | +Click `+GoogleCalendar` at the bottom right to add to your own Google calendar. |
0 commit comments