Make benchmark link count configurable: 10 for PRs, 1000 for main branch#12
Merged
Make benchmark link count configurable: 10 for PRs, 1000 for main branch#12
Conversation
Adding CLAUDE.md with task information for AI processing. This file will be removed when the task is complete. Issue: #11
- Changed LINK_COUNT from a constant to a lazy static that reads from BENCHMARK_LINK_COUNT env var - Defaults to 10 links for faster PR benchmarks - Set to 1000 links for main/master branch via GitHub Actions workflow - Updated all benchmark code to dereference the lazy static with *LINK_COUNT - Applied cargo fmt to format all Rust code Fixes #11 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
This reverts commit b59e757.
Member
Author
🤖 Solution Draft LogThis log file contains the complete execution trace of the AI solution draft process. 💰 Cost estimation:
Now working session is ended, feel free to review and add any feedback on the solution draft. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📋 Summary
This PR implements a configurable benchmark link count to optimize CI/CD performance:
🔧 Implementation Details
Changes Made
Modified
rust/src/lib.rs:LINK_COUNTfrom a constant (const) to a lazy-initialized static that reads from theBENCHMARK_LINK_COUNTenvironment variableonce_cell::Lazyfor thread-safe initializationUpdated benchmark files:
*LINK_COUNTcreate.rs,delete.rs,update.rsModified
.github/workflows/rust.yml:BENCHMARK_LINK_COUNTenvironment variable to the "Run benchmark" step1000when pushing to main or master branch10for all pull requestsCode formatting:
cargo fmtto ensure all Rust code follows the project's formatting standardsBenefits
Testing
cargo check)cargo fmt)📚 Technical Approach
The implementation uses Rust's
Lazy<T>type from theonce_cellcrate to initialize the link count once at runtime by reading the environment variable. This approach:🔗 Related
Fixes #11
🤖 Generated with Claude Code
Co-Authored-By: Claude noreply@anthropic.com