Skip to content

stream: fix performance regression - #39254

Closed
mscdex wants to merge 1 commit into
nodejs:masterfrom
mscdex:streams-fix-perf-regression
Closed

stream: fix performance regression#39254
mscdex wants to merge 1 commit into
nodejs:masterfrom
mscdex:streams-fix-perf-regression

Conversation

@mscdex

Copy link
Copy Markdown
Contributor

Benchmark result:

 confidence improvement accuracy (*) (**) (***)
streams/creation.js kind='duplex' n=50000000 *** 25.65 % ±2.49% ±3.34% ±4.41%

@mscdexmscdex added stream Issues and PRs related to the stream subsystem. performance Issues and PRs related to the performance of Node.js. labels Jul 4, 2021
@github-actionsgithub-actionsBot added the needs-ci PRs that need a full CI run. label Jul 4, 2021
@nodejs-github-bot

This comment has been minimized.

@mscdex
mscdexforce-pushed the streams-fix-perf-regression branch from 3e5aa92 to a1ee30bCompareJuly 4, 2021 11:20
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@mscdexmscdex removed the needs-ci PRs that need a full CI run. label Jul 4, 2021
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@ronag

ronag commented Jul 8, 2021

Copy link
Copy Markdown
Member

Landed in a5ba28d

@ronagronag closed this Jul 8, 2021
ronag pushed a commit that referenced this pull request Jul 8, 2021
PR-URL: #39254
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@mscdex
mscdex deleted the streams-fix-perf-regression branch July 8, 2021 12:02
@BethGriggsBethGriggs mentioned this pull request Oct 14, 2021
2 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

performanceIssues and PRs related to the performance of Node.js.streamIssues and PRs related to the stream subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants

@mscdex@nodejs-github-bot@ronag@jasnell@lpinca@cjihrig@aduh95@ZYSzys@targos