|
| 1 | +# Node.js Performance Team Meeting 2023-07-10 |
| 2 | + |
| 3 | +## Links |
| 4 | + |
| 5 | +* **Recording**: https://www.youtube.com/watch?v=jARbOPW2-I4 |
| 6 | +* **GitHub Issue**: https://github.com/nodejs/performance/issues/101 |
| 7 | + |
| 8 | +## Present |
| 9 | + |
| 10 | +* Performance team: @nodejs/performance |
| 11 | +* Yagiz Nizipli: @anonrig |
| 12 | +* Ulises Gascon: @ulisesGascon |
| 13 | +* Carlos Espa: @Ceres6 |
| 14 | +* Rafael Gonzaga: @RafaelGSS |
| 15 | +* Ruben Bridgewater: @BridgeAR |
| 16 | + |
| 17 | +## Agenda |
| 18 | + |
| 19 | +## Announcements |
| 20 | + |
| 21 | +*Extracted from **performance-agenda** labeled issues and pull requests from the **nodejs org** prior to the meeting. |
| 22 | + |
| 23 | +### nodejs/node |
| 24 | + |
| 25 | +* Improve developer experience for v8 performance related flags [#43407](https://github.com/nodejs/node/issues/43407) |
| 26 | + * This is an issue that started in 2022 and is related to the flags that we use to build Node.js |
| 27 | + * Let’s keep the item in the agenda for now (visibility) |
| 28 | + * Currently this topic is open for community contributions |
| 29 | + |
| 30 | +### nodejs/performance |
| 31 | + |
| 32 | +* What's new for the Node.js Performances? - 2023 version [#98](https://github.com/nodejs/performance/issues/98) |
| 33 | + * The team can publish news in this issue that are later on collected for a Node.js global activity feed in https://github.com/nodejs/nodejs-news-feeder |
| 34 | + * This is a global initiative in the organization |
| 35 | + * Removed from the agenda and discussion going offline |
| 36 | + * This issue will be pinned in the repository |
| 37 | +* Reduce URL initializations in ESM [#92](https://github.com/nodejs/performance/issues/92) |
| 38 | + * It was part of a long process from last month (3 PRs to remove unnecessary urls and parsing content) |
| 39 | + * Open to contributions to finalize the last part of it |
| 40 | +* Improve the performance of package json reader [#91](https://github.com/nodejs/performance/issues/91) |
| 41 | + * Node.js is traversing folders (up to two or three levels) looking for package.json, this is slow |
| 42 | + * There are two PRs already merged authored by Yagiz related to parsing. |
| 43 | + * There is a new benchmark to measure this change (https://github.com/lemire/simdjson_performance_demo) and simdjson library proves that the overhead of reading package json lies in V8 not in Node.js |
| 44 | + * There is an independent problem (loading problem) that is related with v8 and not by simdjson |
| 45 | + * we can discuss with the v8 team regarding json deserialize/serialize potential improvements. This will require some benchmarking to explain the scenario and the potential improvements |
| 46 | + * This is currently open for community contributions |
| 47 | + * This improvement is related to startup, require and json manipulation. This will have a great impact in the community |
| 48 | +* Fast path for `--no-deprecation` [#90](https://github.com/nodejs/performance/issues/90) |
| 49 | + * I think we need a benchmark to show the change of removing deprecations and see the impact of it. The current benchmark shows 1 star confidence, which doesn’t show any improvement. |
| 50 | + * Ruben: I don’t think it’s worth the change, because the code is already optimized as well. |
| 51 | +* Performance of WHATWG ReadableStream.read() [#82](https://github.com/nodejs/performance/issues/82) |
| 52 | +* Expensive `isTraceHTTPEnable` [#81](https://github.com/nodejs/performance/issues/81) |
| 53 | + * There was a consensus that the change recommended in this pull request is not worth the performance impact due to adding the extra-overhead and code complexity. |
| 54 | +* MakeCallback is very slow [#24](https://github.com/nodejs/performance/issues/24) |
| 55 | + |
| 56 | + |
| 57 | + |
| 58 | +## Q&A, Other |
| 59 | + |
| 60 | +## Upcoming Meetings |
| 61 | + |
| 62 | +* **Node.js Foundation Calendar**: https://nodejs.org/calendar |
| 63 | + |
| 64 | +Click `+GoogleCalendar` at the bottom right to add to your own Google calendar. |
| 65 | + |
0 commit comments