forked from nodejs/diagnostics
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
adding minutes from 2018-08-22 meeting (nodejs#222)
- Loading branch information
1 parent
ab4e906
commit ea3acde
Showing
1 changed file
with
57 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,57 @@ | ||
# Node.js Foundation Diagnostics Working Group Meeting 2018-06-13 Minutes | ||
|
||
## Attendees | ||
- @mhdawson | ||
- @mrkmarron | ||
- @mike-kaufman | ||
- @qard | ||
- @ofrobots | ||
- Peter | ||
|
||
|
||
## Links | ||
- **This Meeting**: | ||
- Minutes: https://github.com/nodejs/diagnostics/blob/master/wg-meetings/2018-08-22.md | ||
- Notes: https://docs.google.com/document/d/1b7InhqjUTZY0c9jYrpFgzo0xwDgbZ4LjtLPH98ZU4kg/edit | ||
- You Tube: http://youtu.be/0MJ_VIKzoJg | ||
- **Previous Meeting**: | ||
- Minutes: https://github.com/nodejs/diagnostics/blob/master/wg-meetings/2018-08-08.md | ||
- Notes: https://docs.google.com/document/d/15ehqimizTFdJxzd9B8snP3Qf_b9Y7h3SRGRUXScn8rc/edit | ||
- You Tube:http://youtu.be/Ef4sJNR0mQk | ||
|
||
|
||
## Agenda | ||
|
||
Extracted from **diag-agenda** labelled issues and pull requests from the **nodejs org** prior to the meeting. | ||
|
||
### nodejs/diagnostics | ||
|
||
- Diag WG Deep Dives - topics [#168](https://github.com/nodejs/diagnostics/issues/168) | ||
- @mike-kaufman to schedule deep-dive on async-context formalizations. | ||
|
||
- Expectation about tier of support from diagnostic tools and VMs [#157](https://github.com/nodejs/diagnostics/issues/157) | ||
- @mhdawson has PR open | ||
- markdown failed CI :( | ||
- Peter wants to add v8 Code Event Listener APIs to the list | ||
- suggestion is to let current PR land and then open a new PR to add this to the list | ||
|
||
- \[async_hooks\] stable API - tracking issue [#124](https://github.com/nodejs/diagnostics/issues/124) | ||
- Open PR [21313](github.com/nodejs/node/pull/21313), eliminates need for destroy callback & significantly improves perf | ||
- this is stalled | ||
- perf is still biggest ? | ||
- @ofrobots: if we want something from v8, we need time to let it land | ||
- need to figure out the story in next 3 months if we want this stable for node 12. | ||
|
||
- Async-context formalization and diagnostics support [#107](https://github.com/nodejs/diagnostics/issues/107) | ||
- PR https://github.com/nodejs/diagnostics/pull/197 open, would love feedback | ||
- will schedule a deep-dive | ||
|
||
- \[trace_event\] tracking issue [#84](https://github.com/nodejs/diagnostics/issues/84) | ||
- @mhdawson to ping @jasnell for an update here. | ||
- issue needs cleaned up | ||
|
||
|
||
- \[Diagnostics "Best Practices" Guide?\] [#211](https://github.com/nodejs/diagnostics/issues/211) | ||
- Mike from NodeSource is interested in helping out here. | ||
- There's existing content, just needs some curation & editing | ||
- Also need to interface w/ website team. |