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

古いリモート投稿自動削除設定の除外条件に、ブースト・引用・返信された投稿を含める #627

Open
kmycode opened this issue Mar 3, 2024 · 1 comment

Comments

@kmycode
Copy link
Owner

kmycode commented Mar 3, 2024

挙動を変更してほしい機能や動作

image

「お気に入り・ブックマークされた投稿を削除対象から除外する」設定

現在の挙動

削除対象から除外する投稿の条件

お気に入り/ブックマークされている

変更してほしい新しい挙動

新たに「ローカルユーザーによってBT/引用/返信されている」という条件への対応を追加

必要性

本来は投稿に対してローカルユーザーがリアクションしたものについては削除しないような設定にしたかったが、負荷を監視する必要があることから、まずお気に入り・ブックマークのみを対象とした。

お気に入り・ブックマークは、独自のテーブルで管理されている。
ブースト・引用・返信のデータは投稿一覧のテーブル内に含まれているが、このテーブルは行数がとにかく多く、DBのインデックスつけに失敗した場合は多大な負荷が予想される。
この仕様変更については、DBへの負荷を見ながら慎重に検討する。

@kmycode kmycode added this to the kb12.0 milestone Mar 3, 2024
@kmycode
Copy link
Owner Author

kmycode commented Mar 6, 2024

「お気に入り、ブックマーク」の抽出SQLにパフォーマンス上の懸念が出てきましたので、こちらは保留します

なお当該処理はユーザーや外的要因によらず、常に定期的にしか呼び出されないので直ちに改善が必要なわけではないと認識しています

@kmycode kmycode removed this from the kb12.0 milestone Mar 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant