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

Change how subgraph_deployment is organized #2267

Closed
wants to merge 39 commits into from
Closed

Conversation

lutter
Copy link
Collaborator

@lutter lutter commented Mar 12, 2021

This PR is a steptowards using the same id as the primary key as deployment_schemas. It sits on top of #2266 since the former can easily be rolled back in the hosted service if anything goes wrong. This PR contains an irreversible migration and should therefore be rolled out by itself.

@lutter lutter changed the title Change how subgraph_deloyment is organized Change how subgraph_deployment is organized Mar 12, 2021
@lutter lutter force-pushed the lutter/deployment branch 2 times, most recently from 9013781 to 698cda0 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/deployment branch from 53a7906 to a9e366c Compare March 14, 2021 01:15
@lutter lutter force-pushed the lutter/deployment branch from a9e366c to 01cfd1b 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 2 times, most recently from 7f30603 to 8fc990e Compare March 19, 2021 14:58
lutter added 2 commits March 19, 2021 07:59
The test is now independent of how fast the test runs; that will avoid
spurious failures in CI
@lutter lutter force-pushed the lutter/deployment branch 2 times, most recently from 5fffda2 to 857dec3 Compare March 19, 2021 23:49
dependabot-preview bot and others added 10 commits March 22, 2021 10:17
Bumps [anyhow](https://github.com/dtolnay/anyhow) from 1.0.38 to 1.0.39.
- [Release notes](https://github.com/dtolnay/anyhow/releases)
- [Commits](dtolnay/anyhow@1.0.38...1.0.39)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
Bumps [async-trait](https://github.com/dtolnay/async-trait) from 0.1.41 to 0.1.48.
- [Release notes](https://github.com/dtolnay/async-trait/releases)
- [Commits](dtolnay/async-trait@0.1.41...0.1.48)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
Bumps [tokio-stream](https://github.com/tokio-rs/tokio) from 0.1.3 to 0.1.5.
- [Release notes](https://github.com/tokio-rs/tokio/releases)
- [Commits](tokio-rs/tokio@tokio-stream-0.1.3...tokio-stream-0.1.5)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
Since this struct is used to implement LoadQuery for different types,
a generic name is better suited for it.
@lutter lutter force-pushed the lutter/deployment branch from 857dec3 to 64ccc6b 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/deployment branch from f530511 to 85b72ff Compare March 26, 2021 22:19
dependabot-preview bot and others added 10 commits March 29, 2021 14:02
Bumps [anyhow](https://github.com/dtolnay/anyhow) from 1.0.39 to 1.0.40.
- [Release notes](https://github.com/dtolnay/anyhow/releases)
- [Commits](dtolnay/anyhow@1.0.39...1.0.40)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
When an eth adapter would not return the net version or genesis block hash,
we would retry indefinitely. We now only try for 30s. If we can not get the
net version and genesis block hash during that time, we continue without
it. For networks we have used before, we assume that information has not
changed. For new networks, we fail startup.
We used to compare the net identifier from the database in ChainStore::new
with the net identifier we had previously loaded from the database. We now
correctly check the database state with the state we got from providers.
With that, we also use Site more consistently to identify a subgraph
This change gets rid of the block_range column on subgraph_deployment,
renames id, the column for the deployment's IPFS hash to 'subgraph', and
adds an integer id column whose value will be the same as the id in the
deployment_schemas table in the primary.
@lutter lutter force-pushed the lutter/deployment branch from 85b72ff to af76792 Compare March 29, 2021 17:52
@lutter
Copy link
Collaborator Author

lutter commented Mar 29, 2021

This is merged, but github has trouble tracking this because of the different base branch

@lutter lutter closed this Mar 29, 2021
@lutter lutter deleted the lutter/deployment branch March 29, 2021 18:19
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.

3 participants