Skip to content

benchmark: remove redundant timers benchmark - #13009

Closed
Trott wants to merge 1 commit into
nodejs:masterfrom
Trott:timers-bench
Closed

benchmark: remove redundant timers benchmark#13009
Trott wants to merge 1 commit into
nodejs:masterfrom
Trott:timers-bench

Conversation

@Trott

Copy link
Copy Markdown
Member

The immediate.js benchmark with type set to depth measures the same
thing as set-immediate-depth.js. Remove the redundancy.`

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

timers benchmark

The immediate.js benchmark with `type` set to `depth` measures the same
thing as set-immediate-depth.js. Remove the redundancy.`
@nodejs-github-botnodejs-github-bot added benchmark Issues and PRs related to the benchmark subsystem. timers Issues and PRs related to the timers subsystem / setImmediate, setInterval, setTimeout. labels May 13, 2017
@Trott

Copy link
Copy Markdown
MemberAuthor

@Trott

Copy link
Copy Markdown
MemberAuthor

Landed in f2ba06d

@TrottTrott closed this May 17, 2017
Trott added a commit to Trott/io.js that referenced this pull request May 17, 2017
The immediate.js benchmark with `type` set to `depth` measures the same
thing as set-immediate-depth.js. Remove the redundancy.`
PR-URL: nodejs#13009
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
anchnk pushed a commit to anchnk/node that referenced this pull request May 19, 2017
The immediate.js benchmark with `type` set to `depth` measures the same
thing as set-immediate-depth.js. Remove the redundancy.`
PR-URL: nodejs#13009
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
@jasnelljasnell mentioned this pull request May 28, 2017
@gibfahngibfahn mentioned this pull request Jun 15, 2017
3 tasks
@Trott
Trott deleted the timers-bench branch January 13, 2022 22:45
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

benchmarkIssues and PRs related to the benchmark subsystem.timersIssues and PRs related to the timers subsystem / setImmediate, setInterval, setTimeout.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@Trott@refack@joyeecheung@MylesBorins@nodejs-github-bot