Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
doc: add minutes for meeting 29 Jan 2020
PR-URL: #352 Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
- Loading branch information
Showing
1 changed file
with
49 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,49 @@ | ||
# Node.js Foundation Diagnostics WorkGroup Meeting 2020-01-29 | ||
|
||
## Links | ||
|
||
* **Recording**: https://youtu.be/0z9nU1dosiA | ||
* **GitHub Issue**: https://github.com/nodejs/diagnostics/issues/351 | ||
|
||
## Present | ||
|
||
* Michael Dawson (@mhdawson) | ||
* Richard Lau (@richardlau) | ||
* Giovanny Gongora (@gioyik) | ||
* Gireesh Punathil (@gireeshpunathil) | ||
* @legendecas | ||
* Stephen Belanger | ||
|
||
## Agenda | ||
|
||
## Announcements | ||
|
||
*Extracted from **diag-agenda** labelled issues and pull requests from the **nodejs org** prior to the meeting. | ||
### nodejs/diagnostics | ||
|
||
* reportVersion semantics are not defined [#349](https://github.com/nodejs/diagnostics/issues/349) | ||
* Lots of discussion about pro’s cons of multiple numbers | ||
* Gireesh is going to propose single number option bumped on breaking changes in Github | ||
Issue for additional discussion. | ||
|
||
* Proposal to drive Diagnostics WG initiatives through user journeys [#295](https://github.com/nodejs/diagnostics/issues/295) | ||
* Diagnostics "Best Practices" Guide? [#211](https://github.com/nodejs/diagnostics/issues/211) | ||
* Michael submitted - https://github.com/nodejs/node/pull/31501 on using valgrind. | ||
* This issue could use more reviews as well https://github.com/nodejs/diagnostics/pull/343 | ||
|
||
* \[async_hooks\] stable API - tracking issue [#124](https://github.com/nodejs/diagnostics/issues/124) | ||
* Issue on TSC repo, executive summary for CLS API - https://github.com/nodejs/TSC/issues/807 | ||
* 2 proposals, either one will need to be rebased on AsyncExecutionResource | ||
* AsyncExecutionResource a bug was discovered so investigating that | ||
* Previous performance issues have been resolved | ||
* Summary of 2 | ||
* AsyncContext is closer to the existing CLS modules with a Map | ||
* AsyncLocal just gives you the ability to get/set a value | ||
|
||
## Q&A, Other | ||
|
||
## Upcoming Meetings | ||
|
||
* **Node.js Foundation Calendar**: https://nodejs.org/calendar | ||
|
||
Click `+GoogleCalendar` at the bottom right to add to your own Google calendar. |