Use limit() instead of take() because take() produces needless DISTINCT

[KOBA789]

What

Home Timeline の構築で用いている take() を limit() に置き換えます。

Why

クエリパフェーマンスの向上がねらいです。
投稿 note 数の少ないユーザーばかりをフォローしているユーザーにおいて、特に改善幅が大きいです。

複雑な JOIN を含むクエリで take() を用いると、ベーステーブルの行の増幅を避けるために TypeORM が DISTINCT つきのクエリを自動で発行します。
この DISTINCT つきクエリはパフォーマンスへのインパクトが大きく、行の増幅が起きないクエリでは省略すべきです。
現状の Home Timeline のクエリでは行の増幅が起きるような JOIN はしておらず、省略可能です。
明示的に省略する場合は take() ではなく limit() を使います。
参考: https://github.com/typeorm/typeorm/blob/58fc08840a4a64ca1935391f4709a784c3f0b373/docs/select-query-builder.md#using-pagination

Additional info (optional)

@Ry0taK 念のためテストしていただけると嬉しいです。直近でこの周辺のクエリを触っていたようなのでセカンドオピニオンとして適切だろうと思ってメンションしています。

[Ry0taK]

ローカルで以下のケースをテストしてみましたが動作していそうでした

• フォローしているユーザーの合計ノート数が11件以下/12件以上
• 1投稿しかしていないユーザーを10人フォロー

[EbiseLutica]

コード上は問題ないです