Skip to content
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

fix db fallback with FTT #184

Merged
merged 3 commits into from
Apr 27, 2024
Merged

Conversation

anatawa12
Copy link
Collaborator

What

cherry-picks misskey-dev#13495 (it will fix issue misskey-dev#13488)

DB負荷を大丈夫だろうと判断して取り込むことにした。

Why

#178 何かの原因

Additional info (optional)

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

Copy link

このPRによるapi.jsonの差分

差分はこちら

Get diff files from Workflow Page

@anatawa12 anatawa12 changed the title Pick note fetch ftt fix db fallback with FTT Apr 27, 2024
@anatawa12 anatawa12 merged commit d0aa08b into niri-la:develop Apr 27, 2024
20 checks passed
@anatawa12 anatawa12 deleted the pick-note-fetch-ftt branch April 27, 2024 13:04
@tai-cha
Copy link

tai-cha commented Apr 29, 2024

なですきーでは現行で動いているのですが、にりみすのユーザー規模で動いたときの負荷に関してフィードバックもらえると嬉しいです

@Sayamame-beans
Copy link

負荷的には特に問題無さそうな気もします…?
(複数人が一斉に遡ったりしたら話が違うかもしれませんが)

が、今遡ってたらかなり間が空いた所があって気になってます
Screenshot_20240517-111856~2.png

@tai-cha
Copy link

tai-cha commented Jul 17, 2024

やっぱり実装に抜けがありそう(なですきーでも似たような事象を観測はしているので)

@Sayamame-beans
Copy link

今見ると気にならないので謎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants