B4nan / mikro-orm / 455

100%

push

travis-ci-com

perf: use Domain API instead of async_hooks (#58)

After observing huge performance impact when using async_hooks when concurrent requests are involved, Domain API seems like much better fit for this task. Even though they are deprecated, they will most likely stay till another (stable) solution will be available.

So far async_hooks caused memory leaks even without ORM being involved. Simple express server with async_hooks tracing enabled created dozens of references in CONTEXT map without properly destroying them (the destroy hook was simply not called for those resource).

1381 of 1381 branches covered (100.0%)

Branch coverage included in aggregate %.

7 of 7 new or added lines in 1 file covered. (100.0%)

2609 of 2609 relevant lines covered (100.0%)

3480.76 hits per line