Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix: タイムライン読み込み時のN+1問題 #433

Merged
merged 1 commit into from
Jan 9, 2024

Conversation

kmycode
Copy link
Owner

@kmycode kmycode commented Jan 8, 2024

参照先投稿の中身を見るフィルターが原因

引用ではない参照の中身を見ないように修正するのは簡単だけれど、今回はできるだけ仕様を変えないようにしました
今後の調査次第では、後日、負荷軽減を目的として「引用ではない参照先投稿も確認する」オプション(デフォルトでOFF)を追加する可能性はあります

@kmycode kmycode added bug Something isn't working load 負荷 labels Jan 8, 2024
@kmycode kmycode added this to the kb10.0 milestone Jan 8, 2024
@kmycode kmycode merged commit 616d0c5 into kb_development Jan 9, 2024
36 checks passed
@kmycode kmycode deleted the kbtopic-fix-reference-n1 branch January 9, 2024 00:57
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working load 負荷
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant