-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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(backend): 1つのMisskeyで複数のHTTPサーバプロセスを起動できるように #15398
base: develop
Are you sure you want to change the base?
feat(backend): 1つのMisskeyで複数のHTTPサーバプロセスを起動できるように #15398
Conversation
このPRによるapi.jsonの差分 |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #15398 +/- ##
============================================
+ Coverage 18.81% 38.71% +19.89%
============================================
Files 766 1607 +841
Lines 111337 204744 +93407
Branches 1234 4043 +2809
============================================
+ Hits 20947 79257 +58310
- Misses 89801 124847 +35046
- Partials 589 640 +51 ☔ View full report in Codecov by Sentry. |
open |
特定のユーザにメンションするノートを大量に送り付ける負荷テストをしてみたところ…従来の1プロセスで処理するよりも、2プロセスで処理したほうが2倍近くのパフォーマンスを発揮しました。 しかし、残念ながら3, 4…と増やしてみても、何処かがボトルネックになっていて1→2の時ほど伸びはありませんでした…。 |
※v2025.2.1がそこそこ大きくなりつつあるので、v2025.2.1よりも後を望みます
What
メインプロセスでのみHTTPリクエストを受け付けて処理していたのを、複数のワーカープロセスで処理できるように拡張します。
デフォルト値はこのprによる対応前と同じ動作(=HTTPサーバプロセスは1、ワーカープロセスはclusterLimitの数だけ)をするようになっていますので、configを変えない限り影響はありません。
Why
fix #13662
CPUに負担がかかる処理がちらほらとあり、メインプロセス(HTTPサーバプロセス)の負荷となってしまうのを分散させるため
Additional info (optional)
TODO:
Checklist