Repository navigation

async_hooks: add executionAsyncResource - #30959

Closed

Qard wants to merge 24 commits into

nodejs:masterfrom

Qard:current-resource

Commits on Feb 10, 2020
- async_hooks: add currentResource
  
  mcollina
  authored and
  addaleax
  committed
  dd0d929
- rename to ExecutionAsyncResource
  
  mcollina
  authored and
  addaleax
  committed
  6ab9a58
- benchmark fix
  
  mcollina
  authored and
  addaleax
  committed
  1c4abc1
- store resource on AsyncHooks object
  
  Qard
  authored and
  addaleax
  committed
  2fb5a50
- Update docs to use executionAsyncResource
  
  Qard
  authored and
  addaleax
  committed
  63fcaa5
- Convert execution async resource to use a stack
  
  Qard
  authored and
  addaleax
  committed
  e1a4e88
- Add doc note about use of handle APIs being unsafe
  
  Qard
  authored and
  addaleax
  committed
  b66098d
- Clean up some unnecessary bits
  
  Qard
  authored and
  addaleax
  committed
  b95b514
- Give top-level an empty object as a resource
  
  Qard
  authored and
  addaleax
  committed
  a388de9
- Minor docs improvements
  
  Qard
  authored and
  addaleax
  committed
  b3ecef0
- Use stack emplace rather than push
  
  Qard
  authored and
  addaleax
  committed
  0fd3be8
- Do not push promise resource on init
  
  Qard
  authored and
  addaleax
  committed
  1eb13c5
- Rename test files
  
  Qard
  authored and
  addaleax
  committed
  9381824
- Interact with resource stack directly
  
  Qard
  authored and
  addaleax
  committed
  851e478
- Cleanup some things
  
  Qard
  authored and
  addaleax
  committed
  30f79b2
- Handle object reuse properly
  
  Qard
  authored and
  addaleax
  committed
  621d089
- Add test for object reuse fix
  
  Qard
  authored and
  addaleax
  committed
  c485b39
- Make reused resource storage weak
  
  Qard
  authored and
  addaleax
  committed
  36bcdfa
- Stored resource needs to be strong reference
  
  Qard
  authored and
  addaleax
  committed
  60b33d2
- Reset resource after destroy and add note about strong reference safety
  
  Qard
  authored and
  addaleax
  committed
  48920e3
- Better resource stack clearing
  
  Qard
  authored and
  addaleax
  committed
  1d68999
- Make stronger statement about unsafety of using resource properties or methods and avoid delete keyword
  
  Qard
  authored and
  addaleax
  committed
  a8f00fc
- fixup: make benchmark tests pass
  addaleax
  committed
  1a7243c
Commits on Feb 11, 2020
- fixup: docs review nit
  addaleax
  committed
  3402fac

Navigation Menu

Repository navigation

async_hooks: add executionAsyncResource - #30959

async_hooks: add executionAsyncResource#30959
Qard wants to merge 24 commits into
nodejs:masterfrom
Qard:current-resource

Commits on Feb 10, 2020

async_hooks: add currentResource

rename to ExecutionAsyncResource

benchmark fix

store resource on AsyncHooks object

Update docs to use executionAsyncResource

Convert execution async resource to use a stack

Add doc note about use of handle APIs being unsafe

Clean up some unnecessary bits

Give top-level an empty object as a resource

Minor docs improvements

Use stack emplace rather than push

Do not push promise resource on init

Rename test files

Interact with resource stack directly

Cleanup some things

Handle object reuse properly

Add test for object reuse fix

Make reused resource storage weak

Stored resource needs to be strong reference

Reset resource after destroy and add note about strong reference safety

Better resource stack clearing

Make stronger statement about unsafety of using resource properties or methods and avoid delete keyword

fixup: make benchmark tests pass

Commits on Feb 11, 2020

fixup: docs review nit