Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use a synthetic primary key for subgraph_deployment #2268

Merged
merged 1 commit into from
Mar 31, 2021

Conversation

lutter
Copy link
Collaborator

@lutter lutter commented Mar 12, 2021

This PR switches the primary key for subgraph_deployment to be the same as the pk for deployment_schemas. It does the bare minimum necessary to arrange for that in the database, but does not yet contain the code changes required to move away from identifying deployments internally by their IPFS hash. That will come in a future PR.

This PR sits on top of #2267 since manual intervention is needed in the sharded setup in the hosted service between these two PR's.

@lutter lutter force-pushed the lutter/deployment branch from 3afea3f to 9013781 Compare March 12, 2021 02:32
@lutter lutter force-pushed the lutter/deployment2 branch from 8211bb8 to e1490b5 Compare March 12, 2021 02:32
@lutter lutter force-pushed the lutter/deployment branch from 9013781 to 698cda0 Compare March 12, 2021 03:21
@lutter lutter force-pushed the lutter/deployment2 branch from e1490b5 to 1f4edaf Compare March 12, 2021 03:21
@lutter lutter force-pushed the lutter/deployment branch from 698cda0 to 53a7906 Compare March 13, 2021 00:09
@lutter lutter force-pushed the lutter/deployment2 branch from 1f4edaf to eb958e9 Compare March 13, 2021 00:09
@lutter lutter force-pushed the lutter/deployment branch from 53a7906 to a9e366c Compare March 14, 2021 01:15
@lutter lutter force-pushed the lutter/deployment2 branch from eb958e9 to 04a13ff Compare March 14, 2021 01:15
@lutter lutter force-pushed the lutter/deployment branch from a9e366c to 01cfd1b Compare March 16, 2021 22:27
@lutter lutter force-pushed the lutter/deployment2 branch from 04a13ff to f405522 Compare March 16, 2021 22:27
@tilacog tilacog self-requested a review March 16, 2021 22:56
@lutter lutter force-pushed the lutter/deployment branch from 01cfd1b to 7f30603 Compare March 18, 2021 00:07
@lutter lutter force-pushed the lutter/deployment2 branch from f405522 to 9d58a2b Compare March 18, 2021 00:07
@lutter lutter force-pushed the lutter/deployment branch from 7f30603 to 8fc990e Compare March 19, 2021 14:58
@lutter lutter force-pushed the lutter/deployment2 branch from 9d58a2b to 7e3b776 Compare March 19, 2021 14:58
@lutter lutter force-pushed the lutter/deployment branch from 8fc990e to 5fffda2 Compare March 19, 2021 16:08
@lutter lutter force-pushed the lutter/deployment2 branch from 7e3b776 to 4507eec Compare March 19, 2021 16:08
@lutter lutter force-pushed the lutter/deployment branch from 5fffda2 to 857dec3 Compare March 19, 2021 23:49
@lutter lutter force-pushed the lutter/deployment2 branch from 4507eec to 3060635 Compare March 19, 2021 23:49
@lutter lutter force-pushed the lutter/deployment branch from 857dec3 to 64ccc6b Compare March 24, 2021 00:50
@lutter lutter force-pushed the lutter/deployment2 branch from 3060635 to ae4fb48 Compare March 24, 2021 00:50
@lutter lutter force-pushed the lutter/deployment branch from 64ccc6b to f530511 Compare March 24, 2021 21:33
@lutter lutter force-pushed the lutter/deployment2 branch from ae4fb48 to 6bb91b5 Compare March 25, 2021 02:14
@lutter lutter force-pushed the lutter/deployment branch from f530511 to 85b72ff Compare March 26, 2021 22:19
@lutter lutter force-pushed the lutter/deployment2 branch from 6bb91b5 to 42d1960 Compare March 26, 2021 22:20
@lutter lutter force-pushed the lutter/deployment branch from 85b72ff to af76792 Compare March 29, 2021 17:52
@lutter lutter force-pushed the lutter/deployment2 branch from 42d1960 to ba77f20 Compare March 29, 2021 17:53
@lutter lutter changed the base branch from lutter/deployment to master March 29, 2021 18:18
Make 'id' the primary key on subgraph_deployment so that entries there are
identified with the same id as entries in deployment_schemas
@lutter lutter force-pushed the lutter/deployment2 branch from ba77f20 to 46b19f4 Compare March 30, 2021 23:58
@lutter lutter merged commit 46b19f4 into master Mar 31, 2021
@lutter lutter deleted the lutter/deployment2 branch April 21, 2021 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants