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

Subgraph Composition: Support ethereum host functions in dependant subgraphs #5704

Open
wants to merge 1 commit into
base: krishna/sgc-use-firehose-get-block
Choose a base branch
from

Conversation

incrypto32
Copy link
Member

No description provided.

@incrypto32 incrypto32 changed the base branch from master to krishna/cache-firehose-blocks November 18, 2024 10:14
@incrypto32 incrypto32 added this to the Subgraph Composition milestone Nov 18, 2024
@incrypto32 incrypto32 force-pushed the krishna/sgc-eth-host-fns branch from 9988257 to c54b98e Compare November 19, 2024 12:04
@incrypto32 incrypto32 changed the base branch from krishna/cache-firehose-blocks to krishna/switch-to-using-only-block-ptrs November 19, 2024 12:04
@incrypto32 incrypto32 changed the base branch from krishna/switch-to-using-only-block-ptrs to krishna/sgc-use-firehose-get-block November 19, 2024 12:05
@incrypto32 incrypto32 requested a review from mangas November 19, 2024 12:05
@incrypto32 incrypto32 self-assigned this Nov 25, 2024
@incrypto32 incrypto32 force-pushed the krishna/sgc-eth-host-fns branch 2 times, most recently from c54b98e to 1dd59c3 Compare November 25, 2024 12:14
@maoueh
Copy link
Contributor

maoueh commented Nov 25, 2024

Would this work be re-usable to also enable Ethereum hosts fn on Substreams datasource? This was requested before and when I saw the PR, it felt it could be applied to Substreams data sources too.

@incrypto32
Copy link
Member Author

@maoueh it can be applied but what purpose does it serve? Substreams cannot call these host functions on the graph-node anyways (i believe).

@maoueh
Copy link
Contributor

maoueh commented Nov 26, 2024

@incrypto32

There is two Substreams integration now:

  • EntityChanges where we feed graph-node with the right entity changes (no AssemblyScript involved)
  • Substreams sources where a Subgraph dataSources input is the output module of a particular Substreams, this enables AssemblyScript linear processing.

It's on the second case that it make sense to add it. People have been request it to be able to make eth_call in this mode where they run AssemblyScript and they couldn't make eth_call.

@incrypto32
Copy link
Member Author

@maoueh got it, yes i think it makes sense to add them and i believe there's not much additional work other than just adding them as host_fns. Feel free to open an issue.

@maoueh
Copy link
Contributor

maoueh commented Dec 6, 2024

#5727

@incrypto32 incrypto32 force-pushed the krishna/sgc-use-firehose-get-block branch from f95df0a to a1ed8b2 Compare December 10, 2024 10:50
@incrypto32 incrypto32 force-pushed the krishna/sgc-eth-host-fns branch from 1dd59c3 to e0a65dd Compare December 10, 2024 10:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants