Skip to content
This repository was archived by the owner on Feb 6, 2026. It is now read-only.

feat(sdf-server, dal): Adding an edge for default schema variants at pkg import time#3424

Merged
si-bors-ng[bot] merged 3 commits intomainfrom
WIP-Adding-default-schema-variants
Mar 22, 2024
Merged

feat(sdf-server, dal): Adding an edge for default schema variants at pkg import time#3424
si-bors-ng[bot] merged 3 commits intomainfrom
WIP-Adding-default-schema-variants

Conversation

@stack72
Copy link
Copy Markdown
Contributor

@stack72 stack72 commented Mar 12, 2024

Schema / Schema Variant Edges now exist as follows:

DEFAULT: Schema -> Uses -> Schema Variant

In order to make a schema variant the default for a schema, we need to update the
correct edge from Use to Default:

UPDATE: Schema -> Default -> Schema Variant

Therefore, when we are setting a default schema variant we need to find any existing
Default edges and convert them back to uses AND we need to find the existing Use edge
between our nodes and change that to be a Default

@github-actions github-actions Bot added A-sdf Area: Primary backend API service [Rust] A-dal A-rebaser labels Mar 12, 2024
@stack72 stack72 force-pushed the WIP-Adding-default-schema-variants branch 3 times, most recently from 2a7a446 to 553b46d Compare March 14, 2024 14:42
@stack72 stack72 force-pushed the WIP-Adding-default-schema-variants branch from 553b46d to b5b608c Compare March 16, 2024 06:09
@stack72 stack72 force-pushed the WIP-Adding-default-schema-variants branch from b5b608c to 2cfcc5b Compare March 16, 2024 07:10
@stack72 stack72 force-pushed the WIP-Adding-default-schema-variants branch 3 times, most recently from aac57b2 to c6006c9 Compare March 16, 2024 07:18
@stack72 stack72 marked this pull request as ready for review March 16, 2024 07:18
@stack72 stack72 changed the title WIP: Adding an edge for default schema variants at pkg import time feat(sdf-server, dal): Adding an edge for default schema variants at pkg import time Mar 16, 2024
@stack72 stack72 force-pushed the WIP-Adding-default-schema-variants branch 2 times, most recently from 8c45331 to 3c7608f Compare March 16, 2024 07:25
@stack72
Copy link
Copy Markdown
Contributor Author

stack72 commented Mar 16, 2024

bors try

si-bors-ng Bot added a commit that referenced this pull request Mar 16, 2024
@si-bors-ng
Copy link
Copy Markdown
Contributor

si-bors-ng Bot commented Mar 16, 2024

try

Build succeeded:

@stack72 stack72 force-pushed the WIP-Adding-default-schema-variants branch 2 times, most recently from 38bc10a to d5fe89b Compare March 21, 2024 23:09
@stack72
Copy link
Copy Markdown
Contributor Author

stack72 commented Mar 21, 2024

bors try

si-bors-ng Bot added a commit that referenced this pull request Mar 21, 2024
@si-bors-ng
Copy link
Copy Markdown
Contributor

si-bors-ng Bot commented Mar 21, 2024

try

Build succeeded:

@stack72 stack72 requested a review from jhelwig March 21, 2024 23:40
@zacharyhamm
Copy link
Copy Markdown
Contributor

@stack72 pushed up a change: the Use edge is so common that we should probably have a simpler name for the method that creates it. But also we need to include the fact that the edge is default in the merkle tree hash calculation. Looks great to me though!

stack72 added 2 commits March 22, 2024 17:23
…pkg import time

Schema / Schema Variant Edges now exist as follows:

DEFAULT: Schema -> Uses -> Schema Variant

In order to make a schema variant the default for a schema, we need to update the
correct edge from Use to Default:

UPDATE: Schema -> Default -> Schema Variant

Therefore, when we are setting a default schema variant we need to find any existing
Default edges and convert them back to uses AND we need to find the existing Use edge
between our nodes and change that to be a Default
@stack72 stack72 force-pushed the WIP-Adding-default-schema-variants branch 2 times, most recently from 2c9cdf3 to 2b6b2b6 Compare March 22, 2024 17:24
@stack72
Copy link
Copy Markdown
Contributor Author

stack72 commented Mar 22, 2024

bors merge

si-bors-ng Bot added a commit that referenced this pull request Mar 22, 2024
3424: feat(sdf-server, dal): Adding an edge for default schema variants at pkg import time r=stack72 a=stack72

Schema / Schema Variant Edges now exist as follows:

DEFAULT: Schema -> Uses -> Schema Variant

In order to make a schema variant the default for a schema, we need to update the
correct edge from Use to Default:

UPDATE: Schema -> Default -> Schema Variant

Therefore, when we are setting a default schema variant we need to find any existing
Default edges and convert them back to uses AND we need to find the existing Use edge
between our nodes and change that to be a Default

Co-authored-by: stack72 <public@paulstack.co.uk>
Co-authored-by: Zachary Hamm <zack@systeminit.com>
@si-bors-ng
Copy link
Copy Markdown
Contributor

si-bors-ng Bot commented Mar 22, 2024

Build failed:

…the edge info in the merkle tree hash calculation
@stack72 stack72 force-pushed the WIP-Adding-default-schema-variants branch from 2b6b2b6 to 07fb3a7 Compare March 22, 2024 17:48
@stack72
Copy link
Copy Markdown
Contributor Author

stack72 commented Mar 22, 2024

bors merge

@si-bors-ng
Copy link
Copy Markdown
Contributor

si-bors-ng Bot commented Mar 22, 2024

Build succeeded:

@si-bors-ng si-bors-ng Bot merged commit 3fc65d7 into main Mar 22, 2024
@si-bors-ng si-bors-ng Bot deleted the WIP-Adding-default-schema-variants branch March 22, 2024 18:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

A-dal A-sdf Area: Primary backend API service [Rust]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants