Concept detail with `includeInverseMappings=true` returns every inverse mapping, unpaginated

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

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

評価

難易度
4/5
見積もり時間
3〜5日
初心者へのやさしさ
45/100
issue の種類
機能追加
明瞭さ
おおむね明確
活発さ
活発
領域
api

調査の方向性

Compare the concept detail endpoint with the mappings list URL described in the issue to understand how inverse mappings are included and paginated. First clarify the proposed response behavior with the TBv2 and TBv3 owners; then check the endpoint documentation and relevant API tests. Done means the agreed cap, total and paging URL (or caller guidance) are covered, with tests for the large result set.

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

説明

component/api decision/human-review program/platform-health signal/needs-human signal/small-scope stage/triaged type/bug

GET /orgs/CIEL/sources/CIEL/concepts/1065/?includeMappings=true&includeInverseMappings=true returns 869 mappings (1.39 MB) and takes 1.2–2.5 s. Several LOINC concepts, e.g. 72230-6 and LA6577-6, have 6,000–7,200 inverse mappings in their own source, so their responses would run to many MB.

The repository's mappings list, …/mappings/?toConcept=…&toConceptSource=…&toConceptOwner=…, already pages and counts the same set, in about 0.3 s.

  • Concept detail caps the inverse mappings it includes inline (e.g. at 100), and reports the total and the list URL that pages through the rest. Or the docs point callers to the list.
  • The change to default behaviour is agreed with the TBv2 and TBv3 owners.
主要言語
言語のデータがありません
スター
4
フォーク
2
平均マージ
2時間 12分
マージ済み PR(30日)
1

環境構築

はじめの一歩

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

OpenConceptLab/ocl_issues のほかの issue

OpenConceptLab/ocl_issues の issue をすべて見る

似ている issue

Backend & API Design の issue をもっと見る

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

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