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

ソーシャルタイムライン等をたくさん遡るとノートの取得漏れが起きることがある #13488

Open
1 task done
tai-cha opened this issue Mar 1, 2024 · 1 comment · May be fixed by #13495
Open
1 task done
Assignees
Labels
⚠️bug? This might be a bug

Comments

@tai-cha
Copy link
Contributor

tai-cha commented Mar 1, 2024

💡 Summary

ソーシャルTLはhomeTimelineとlocalTimelineの合成で、現在redisにキャッシュしたノートidをベースに取得している

homeTimelineかlocalTimelineのうち、古い方のノートidまでが一回のレスポンスに含まれるが、HTLとLTLではキャッシュの保存件数も違うため、キャッシュに含まれていない部分が正常にフォールバックされない(キャッシュの件数にも上限がある)

(現在は取得したノートidのリストをArray.flatten(1)で単純に合成しているため、この現象が起こる)

もしかしたらwithFilesなど複数のキーからノートidを合成して作成するTLでは同じ現象が起きるかもしれない

🥰 Expected Behavior

HTL、STLのノートがそれぞれ正常にDBにフォールバックされる

🤬 Actual Behavior

STLのノートの一部(ホーム投稿のノート)などが取得漏れを起こす

📝 Steps to Reproduce

  1. 何らかのパブリック投稿を行う
  2. 何らかのホーム投稿を行う
  3. reidisの直接操作などでhomeTimelineのキーに対するキャッシュを破棄する
  4. STLを参照するとLTL相当のノートしか見えなくなる

💻 Frontend Environment

* Model and OS of the device(s):
* Browser:
* Server URL:
* Misskey: 2024.2.0

🛰 Backend Environment (for server admin)

* Installation Method or Hosting Service:
* Misskey: 2024.2.0
* Node:
* PostgreSQL:
* Redis:
* OS and Architecture:

Do you want to address this bug yourself?

  • Yes, I will patch the bug myself and send a pull request
@tai-cha tai-cha added the ⚠️bug? This might be a bug label Mar 1, 2024
@tai-cha tai-cha changed the title ソーシャルタイムラインをとても遡ると表示漏れが起きる ソーシャルタイムラインをたくさん遡ると表示漏れが起きる Mar 1, 2024
@tai-cha tai-cha changed the title ソーシャルタイムラインをたくさん遡ると表示漏れが起きる ソーシャルタイムラインをたくさん遡ると表示漏れが起きることがある Mar 1, 2024
@tai-cha
Copy link
Contributor Author

tai-cha commented Mar 2, 2024

すこし分かりにくいので図解(横軸はノートの作成時刻)
IMG_0274

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚠️bug? This might be a bug
Projects
1 participant