Skip to content

Commit

Permalink
Merge pull request #54 from ga4gh/no-metadata-endpoint
Browse files Browse the repository at this point in the history
Add ADR for no metadata endpoint
  • Loading branch information
nsheff authored Jul 27, 2023
2 parents cd8d83a + b6f8785 commit dcfecc3
Showing 1 changed file with 23 additions and 0 deletions.
23 changes: 23 additions & 0 deletions docs/decision_record.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,28 @@ The key words "MUST", "MUST NOT", "REQUIRED", "SHALL", "SHALL NOT", "SHOULD", "S
[TOC]


## 2023-07-26 There will be no metadata endpoint

### Decision

We have no need for a `/metadata` endpoint

### Rationale

At one point (issue #3), we debated whether there should be a `/metadata` endpoint or something like that as a way to retrieve information about a sequence that might not be part of the digested sequence. However, after we distinguised between `inherent` and `non-inherent` attributes, we have realized that this satisifes the earlier requirement for a `/metadata` endpoint; in fact, the metadata can be returned to the user through the normal endpoint, and just flagged as `non-inherent` in the schema to indicate that it's not digested, and therefore not part of the identity of the object

We distinguished between two types of metadata:

- server-scoped metadata, like the schema we described above, should be served by `/service-info`
- collection-scoped or sequence-scoped metadata don't fit under `/service-info`. For these, they will be served by the primary `/collection` endpoint, rather than by a separate `/metadata` endpoint.

### Linked issues

- https://github.com/ga4gh/seqcol-spec/issues/3
- https://github.com/ga4gh/seqcol-spec/issues/39
- https://github.com/ga4gh/seqcol-spec/issues/40


## 2023-07-12 Implementations SHOULD provide sorted_name_length_pairs and comparison endpoint

### Decisions
Expand Down Expand Up @@ -68,6 +90,7 @@ We see no need to add prefixes to the identifiers we use internally, which we ju

- https://github.com/ga4gh/seqcol-spec/issues/37


## 2023-06-28 Details of endpoints

### Decisions
Expand Down

0 comments on commit dcfecc3

Please sign in to comment.