-
Notifications
You must be signed in to change notification settings - Fork 92
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: enable asyncLocalStorage by default #251
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Requests/sec: 17395.81 | ||
Transfer/sec: 12.04MB | ||
|
||
v12.22.12 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
v12 性能是真的下降很多。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
毕竟那个版本还没优化 async hook 吧,反正不是我们最小支持版本了
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
毕竟那个版本还没优化 async hook 吧,反正不是我们最小支持版本了
春节之后给 egg-core 发布一个 major 版本,只支持 node 14+
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #251 +/- ##
=======================================
Coverage 99.75% 99.75%
=======================================
Files 19 19
Lines 1206 1207 +1
Branches 211 211
=======================================
+ Hits 1203 1204 +1
Misses 3 3 ☔ View full report in Codecov by Sentry. |
4.29.0 |
koajs/koa#1721