graphprotocol / graphprotocol/graph-node

[Bug] Disallow sorting by child fields that are arrays

オープン
#5,207 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

bug Stale
主要言語
Rust
スター
3.2k
フォーク
1.1k
平均マージ
4日 1時間
マージ済み PR(30日)
1

説明

Bug report

In a subgraph schema like

type Parent {
  id: ID!
  children: [Child!]!
}

we allow a query like parents(orderBy: children__id) { .. } but that ordering doesn't make much sense. When a parent has multiple children, it's not clear which child's id we will use for the ordering. We should simply not support that.

Some information to help us out
  • Tick this box if this bug is caused by a regression found in the latest release.
  • Tick this box if this bug is specific to the hosted service.
  • I have searched the issue tracker to make sure this issue is not a duplicate.
OS information

None

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

issue にはファイル、テスト、エントリポイントが指定されていません。まず、ネストされたリレーションに対して GraphQL orderBy フィールドがどのように検証されるかを追跡し、次に Parent/children スキーマのケースを再現してください。配列値の子フィールドによる並べ替えが拒否され、サポートされているスカラーリレーションは引き続き有効であれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
graphql, rust
領域
api
issue の種類
バグ
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
45/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。