Pick redis push tl #75
Conversation
anatawa12
force-pushed
the
pick-redis-push-tl
branch
from
ba2a022 to
c941eae
Compare
github-actions
bot
removed
packages/frontend
packages/backend:test
packages/misskey-js
labels
|
誰かレビューしてほしい |
|
misskey-dev#12001 の修正である 7a8d5e5 の一部も取り込む必要あり |
|
変更シンプルにしたいのでuserごとのTLのキャッシュの長さの制限一時的に(2023.10.xマージまで)固定にしたい。(metaにmigrationつけたくない) @niwaniwa @Sayamame-beans いい？ |
|
別に支障ないだろうし良いんじゃないですかね(適当) |
This reverts commit c74fff1.
| name = 'WithReplies1696222183852' | ||
|
|
||
| async up(queryRunner) { | ||
| await queryRunner.query(`ALTER TABLE "following" ADD "withReplies" boolean NOT NULL DEFAULT false`); |
There was a problem hiding this comment.
まだここのdefault値で悩んでます。なんか切り替える機能を実装するしかないかな
Misskeyコードのレビューは厳しい…かも… |
…_list_joining -> user_list_membership
|
RedisへのTLの構築をListで行うようにの中で行われたrefactorをrevertすることでdiffが減った。 |
anatawa12
force-pushed
the
pick-redis-push-tl
branch
from
ad9c167 to
fefdb4b
Compare
|
センシティブチャネルの扱いがまだこのサーバに合わせてないので合わせる必要あり |
|
うーん、そもそもupstreamでチャンネル投稿含める設定だとセンシティブも含まれるようになってそうだな。なのでRedis的には今のままでいいや。エンドポイントの方でどうにかする |
|
いやあとから分けるほうが大変だから分けといたほうがいいか...? |
|
upstreamではredisから取った後にフィルタって実装なのでそのままでいいや。 |
|
redisへの追加条件を2023.10.1にできる限り近づけた。 |
|
自己レビューはOK出します。 |
|
そういえば遡れるnote数このままでいいのかな。少なくて一日も遡れないとかにならないか確認しないと |
|
考えないといけなそうなupstreamのissueが生えた misskey-dev#12027 |
|
いや misskey-dev#12027 は現行と同じ動作だからいいや |
|
頑張ればredis to redisのアンテナマイグレーションできそうだけど別トピック |
|
とりあえずこれ自体は問題ないと思うのでマージ |
What
Why
Additional info (optional)
Checklist