-
-
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
enhance(backend): migrate bull to bullmq #10910
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #10910 +/- ##
===========================================
+ Coverage 77.25% 77.34% +0.08%
===========================================
Files 740 740
Lines 70373 70409 +36
Branches 6402 6423 +21
===========================================
+ Hits 54369 54459 +90
+ Misses 16004 15950 -54
|
ERR Lua redis lib command arguments must be strings or integers script: 2eddb0e6d7479fa4b1e57e8d03beec511bc502ea, on @user_script:192. |
ヌァァァァァァァァァァァァァァァァァァァァァァァァァァァァァァァァァァンンンンオオオオンンオンオンオンオンンンンンンンンン゛ン゛!!!!!!!!!!!!!!!! |
テストは通った |
move e2eが通ってるなら完全に動作していると言って過言ではなさそう |
マージするか |
何もいじってないのに全く関係ないとこでエラー出てきた? |
関係ないことはなかったけどなぜそうなるのかは謎 |
92ef167 時点では通ってるし |
その後のコミットでバックエンドは変更してないのでなぜエラーが出るようになったのかも謎 |
f4dc7ad の意図は何? |
簡単にエラーオブジェクトに置き換えられなそうだったのと、どこからも使われてなかったからとりあえずコメントアウトした |
文字列throwをやめたのは、bull-mqだとその方法でのthrowは対応してないらしいからついでに全部書き換えた |
いや普通に使われてるかも |
使われてる |
そして普通に通った |
@tamaina 🙏🏻🙏🏻🙏🏻 |
連合も問題なさそう |
Bull dashboardで良い感じに表示されるようになった |
* wip * wip * Update QueueService.ts * wip * refactor * ✌️ * fix * Update QueueStatsService.ts * refactor * Update ApNoteService.ts * Update mock-resolver.ts * refactor * Update mock-resolver.ts
This reverts commit fd7b77c.
Revert "enhance(backend): migrate bull to bullmq (misskey-dev#10910)"
This reverts commit fd7b77c.
This reverts commit fd7b77c.
What
Resolve #6221
Why
See #6221
Additional info (optional)
Checklist