Skip to content

Conversation

kfarnung
Copy link
Contributor

@kfarnung kfarnung commented Jan 22, 2018

Update performance_state to use AliasedBuffer and update usage sites.

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

http2, perf_hooks

@kfarnung kfarnung requested review from addaleax and jasnell January 22, 2018 22:19
@nodejs-github-bot nodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. lib / src Issues and PRs related to general changes in the lib or src directory. labels Jan 22, 2018
@kfarnung
Copy link
Contributor Author

/cc @mike-kaufman @digitalinfinity

@addaleax addaleax added http2 Issues or PRs related to the http2 subsystem. perf_hooks Issues and PRs related to the implementation of the Performance Timing API. labels Jan 22, 2018
@kfarnung
Copy link
Contributor Author

@kfarnung
Copy link
Contributor Author

kfarnung commented Jan 23, 2018

CI rerun: https://ci.nodejs.org/job/node-test-pull-request/12689/

Looks like the Windows leg is hitting #18269. Everything else is passing.

@kfarnung
Copy link
Contributor Author

@jasnell do you have a moment to take a look?

@kfarnung kfarnung self-assigned this Jan 24, 2018
@kfarnung
Copy link
Contributor Author

kfarnung commented Jan 24, 2018

Final (hopefully) rerun: https://ci.nodejs.org/job/node-test-pull-request/12712/

Looks like there are a couple of unrelated issues.

@addaleax addaleax added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jan 24, 2018
@kfarnung kfarnung closed this Jan 25, 2018
@kfarnung kfarnung merged commit be2cbcc into nodejs:master Jan 25, 2018
@kfarnung
Copy link
Contributor Author

Landed in be2cbcc

@kfarnung
Copy link
Contributor Author

I created a backport PR #18364

kfarnung added a commit to kfarnung/node-chakracore that referenced this pull request Jan 25, 2018
Update performance_state to use AliasedBuffer and update usage sites.

PR-URL: nodejs/node#18300
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: James M Snell <[email protected]>
kfarnung added a commit to kfarnung/node-chakracore that referenced this pull request Jan 25, 2018
Update performance_state to use AliasedBuffer and update usage sites.

PR-URL: nodejs/node#18300
Backport-PR-URL: nodejs#453
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Mike Kaufman <[email protected]>
evanlucas pushed a commit that referenced this pull request Jan 30, 2018
Update performance_state to use AliasedBuffer and update usage sites.

PR-URL: #18300
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: James M Snell <[email protected]>
kfarnung added a commit to kfarnung/node that referenced this pull request Feb 2, 2018
Update performance_state to use AliasedBuffer and update usage sites.

PR-URL: nodejs#18300
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@addaleax addaleax removed the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Feb 4, 2018
gibfahn pushed a commit that referenced this pull request Feb 19, 2018
Update performance_state to use AliasedBuffer and update usage sites.

PR-URL: #18300
Backport-PR-URL: #18364
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: James M Snell <[email protected]>
kjin pushed a commit to kjin/node that referenced this pull request May 1, 2018
This is the portion of be2cbcc that is not in dea44b9.

Update performance_state to use AliasedBuffer and update usage sites.

PR-URL: nodejs#18300
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: James M Snell <[email protected]>
MylesBorins pushed a commit that referenced this pull request May 2, 2018
This is the portion of be2cbcc that is not in dea44b9.

Update performance_state to use AliasedBuffer and update usage sites.

Backport-PR-URL: #20456
PR-URL: #18300
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@MylesBorins MylesBorins mentioned this pull request May 2, 2018
MayaLekova pushed a commit to MayaLekova/node that referenced this pull request May 8, 2018
Update performance_state to use AliasedBuffer and update usage sites.

PR-URL: nodejs#18300
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++ Issues and PRs that require attention from people who are familiar with C++. http2 Issues or PRs related to the http2 subsystem. lib / src Issues and PRs related to general changes in the lib or src directory. perf_hooks Issues and PRs related to the implementation of the Performance Timing API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants