Commit b30c40b
perf_hooks: import http2 only once
Import http2 internal bindings on each collectHttp2Stats call is not
required.
PR-URL: #29419
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Minwoo Jung <minwoo@nodesource.com>1 parent 844b45b commit b30c40b
1 file changed
Lines changed: 2 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -92,11 +92,12 @@ const IDX_SESSION_STATS_DATA_SENT = 6; | |||
| 92 | 92 | const IDX_SESSION_STATS_DATA_RECEIVED = 7; | |
| 93 | 93 | const IDX_SESSION_STATS_MAX_CONCURRENT_STREAMS = 8; | |
| 94 | 94 | ||
| 95 | + let http2; | ||
| 95 | 96 | let sessionStats; | |
| 96 | 97 | let streamStats; | |
| 97 | 98 | ||
| 98 | 99 | function collectHttp2Stats(entry) { | |
| 99 | - const http2 = internalBinding('http2'); | ||
| 100 | + if (http2 === undefined) http2 = internalBinding('http2'); | ||
| 100 | 101 | switch (entry.name) { | |
| 101 | 102 | case 'Http2Stream': | |
| 102 | 103 | if (streamStats === undefined) | |
0 commit comments