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

Graph Program V3 #7

Draft
wants to merge 2 commits into
base: dev
Choose a base branch
from
Draft

Graph Program V3 #7

wants to merge 2 commits into from

Conversation

kevinrodriguez-io
Copy link
Contributor

@kevinrodriguez-io kevinrodriguez-io commented Dec 10, 2022

** DO NOT MERGE **

This PR includes a proposal to move to Account resizing and avoiding PDA spam by creating a single Storage Account.

Pros:

  • Simpler structure
  • Probably more performant at Sealevel
  • Easier to Index
  • Easier to add bulk follow IXs in the future
  • Easier to cache client-side

Cons:

  • Anchor not playing nice with HashSets yet so need to deal with Vec searching back and forth (Probably worth to add manual de/ser) or try beet by Metaplex
  • Bulky reads of unnecessary data sometimes

Pending:

  • Create migration scripts, IXs from V2->V3
  • Use HashSet structure instead of Vectors or start looking at Merkle roots

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.

1 participant