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

Update to Solana 1.16 and use new Bubblegum Rust client #127

Merged
merged 7 commits into from
Oct 15, 2023

Conversation

danenbm
Copy link
Contributor

@danenbm danenbm commented Oct 9, 2023

Overview

Updates DAS API to use Solana 1.16 versions, plus the initial changes for Bubblegum Update Metadata (uses the Bubblegum Rust client that supports Update Metadata, but doesn't actually do anything with it yet).

Details

  • Change path for load generation
  • Update to Solana 1.16 versions
    • Update to Solana 1.16.16 (version that does not have tokio pinned and solana-transaction-status supports Borsh 0.10.3).
    • Update blockbuster to latest branch that supports Bubblegum Update Metadata instruction parsing (still a commit ID).
    • Update to Borsh 0.10.3.
    • Update to Plerkle Messenger 1.6.0.
    • Update to Bubblegum 1.0.1-beta.2 (Rust client version that supports Update Metadata and limits to Borsh < 1.0.0).
    • Update to SPL Account Compression 0.2.0.
    • Update to Anchor 0.28.0.
    • Update Candy Machine and Candy Guard to 2.0.x.
    • spl-associated-token-account and spl-token use same version ranges as Bubblegum program crate.
    • Update SPL No-op.
    • MPL Token Metadata exact version: 2.0.0-beta.1.

* Update to Solana 1.16.16 (version that does not have
tokio pinned and solana-transaction-status supports
Borsh 0.10.3).
* Update blockbuster to latest branch that supports
Bubblegum Update Metadata instruction parsing (still
a commit ID).
* Update to Borsh 0.10.3.
* Update to Plerkle Messenger 1.6.0.
* Update to Bubblegum 1.0.1-beta.2 (Rust client version
that supports Update Metadata and limits to Borsh < 1.0.0).
* Update to SPL Account Compression 0.2.0.
* Update to Anchor 0.28.0.
* Update Candy Machine and Candy Guard to 2.0.x.
* spl-associated-token-account and spl-token use same version
ranges as Bubblegum program crate.
* Update SPL No-op.
* MPL Token Metadata exact version: 2.0.0-beta.1.
@danenbm danenbm changed the base branch from main to update-metadata-staging October 9, 2023 19:09
@danenbm danenbm changed the title Danenbm/update metadata Bubblegum Update Metadata Oct 9, 2023
@danenbm danenbm changed the title Bubblegum Update Metadata Update to Solana 1.16 and use new Bubblegum Rust client Oct 9, 2023
@danenbm danenbm marked this pull request as ready for review October 9, 2023 23:55
Load.Dockerfile Outdated Show resolved Hide resolved
@danenbm danenbm force-pushed the danenbm/update-metadata branch from b8214bf to fbaae25 Compare October 13, 2023 16:58
@danenbm danenbm changed the base branch from update-metadata-staging to main October 13, 2023 21:16
@danenbm danenbm requested a review from fanatid October 13, 2023 21:27
@@ -71,17 +71,15 @@ impl IngesterConfig {
}

pub fn get_account_backfill_stream_worker_count(&self) -> u32 {
self.account_backfill_stream_worker_count
.unwrap_or_else(|| self.get_account_stream_worker_count())
self.account_backfill_stream_worker_count.unwrap_or(0)
Copy link
Contributor Author

@danenbm danenbm Oct 13, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have to set this to zero for now because we are not configuring any backfill streams in the plugin. @fanatid will touch base next week since we previously talked about the config, and we can see what is good solution.

}

pub fn get_transaction_stream_worker_count(&self) -> u32 {
self.transaction_stream_worker_count.unwrap_or(2)
}

pub fn get_transaction_backfill_stream_worker_count(&self) -> u32 {
self.transaction_backfill_stream_worker_count
.unwrap_or_else(|| self.get_transaction_stream_worker_count())
self.transaction_backfill_stream_worker_count.unwrap_or(0)
Copy link
Contributor Author

@danenbm danenbm Oct 13, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have to set this to zero for now because we are not configuring any backfill streams in the plugin. @fanatid will touch base next week since we previously talked about the config, and we can see what is good solution.

@danenbm danenbm merged commit 8ca22ca into main Oct 15, 2023
2 checks passed
@danenbm danenbm deleted the danenbm/update-metadata branch October 15, 2023 00:16
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