Skip to content

Conversation

@nolanmar511
Copy link
Contributor

@nolanmar511 nolanmar511 commented May 2, 2019

Reverts #476

#476 should not have been merged in.

It allows the profiler to run with versions of Node 10 impacted by https://bugs.chromium.org/p/chromium/issues/detail?id=847863 (gc can take several minutes when heap profiling is enabled)

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label May 2, 2019
@nolanmar511 nolanmar511 requested review from bcoe and kalyanac May 2, 2019 19:02
Copy link
Contributor

@JustinBeckwith JustinBeckwith left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apologies!

@kalyanac kalyanac merged commit 2453786 into master May 2, 2019
@nolanmar511 nolanmar511 deleted the revert-476-update-engine-field-2-the-updates-revenge branch May 2, 2019 19:45
@nolanmar511
Copy link
Contributor Author

No problems!

FYI to bcoe@ -- PR#476 has been reverted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants