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

Support Geyser plugin #48

Open
Aursen opened this issue Mar 30, 2024 · 5 comments
Open

Support Geyser plugin #48

Aursen opened this issue Mar 30, 2024 · 5 comments
Labels
enhancement New feature or request

Comments

@Aursen
Copy link
Collaborator

Aursen commented Mar 30, 2024

At the moment, we have no way of using geyser plugins with LiteSVM. This may be useful for testing programs like Clockwork. Put it behind a feature flag

@Aursen Aursen added the enhancement New feature or request label Mar 30, 2024
@kevinheavey
Copy link
Collaborator

I am not sure what this would involve but it sounds like a lot of work.

Also if possible I strongly recommend a second crate composing on top of litesvm instead of a feature flag

@sonicfromnewyoke
Copy link

IMO, this is not a huge task.
We have to implement a lite-weight version of the plugin-manager and slightly modify the existing code (accounts_db.rs for update_account notification and so on). I'd like to take part in the realization of this enhancement, and maybe split it into a couple of deployable steps, like:

  1. plugin-manager
  2. accounts
  3. txs
  4. slots

@Aursen
Copy link
Collaborator Author

Aursen commented Sep 13, 2024

IMO, this is not a huge task. We have to implement a lite-weight version of the plugin-manager and slightly modify the existing code (accounts_db.rs for update_account notification and so on). I'd like to take part in the realization of this enhancement, and maybe split it into a couple of deployable steps, like:

  1. plugin-manager
  2. accounts
  3. txs
  4. slots

Amazing to know, a lot of plugin use the local client too. Maybe can we integrate a lite server for this case too

@sonicfromnewyoke
Copy link

@Aursen, yeah, you mean having an opportunity to fetch get_balance, get_account, and so on, that are pointing to the HashMap<Pubkey, AccountSharedData> instead of Banks fork, right?

@Aursen
Copy link
Collaborator Author

Aursen commented Sep 13, 2024

@Aursen, yeah, you mean having an opportunity to fetch get_balance, get_account, and so on, that are pointing to the HashMap<Pubkey, AccountSharedData> instead of Banks fork, right?

Yep that's the idea

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants