Skip to content

Small changes for amp subgraphs#6379

Open
lutter wants to merge 2 commits intomasterfrom
lutter/amp-name
Open

Small changes for amp subgraphs#6379
lutter wants to merge 2 commits intomasterfrom
lutter/amp-name

Conversation

@lutter
Copy link
Collaborator

@lutter lutter commented Feb 16, 2026

This PR makes two somewhat unrelated changes:

  • It's now possible to configure a different network name for amp than what graph-node uses (e.g., mainnet vs ethereum-mainnet) For amp-powered subgraphs, it is expected that the manifest uses the amp name
  • Simplify the naming of CTEs in amp queries to make them a little more readable

AMP-powered subgraphs use different network names than graph-node's
internal chain names (e.g., AMP uses "ethereum-mainnet" while graph-node
uses "mainnet"). Add a config-level `amp` field on chains that maps AMP
names to internal names, so AMP manifests work without renaming chains
in the database.
@lutter lutter requested a review from isum February 16, 2026 23:27
The old convention made queries very hard to read. We now simply demand
that users stay away from tables etc. whose name starts with `amp_`
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