Skip to content

Conversation

gregorydemay
Copy link
Member

@gregorydemay gregorydemay commented Sep 12, 2025

Follow-up on #6706 to benchmark the Dogecoin adapter when adding 800k Dogecoin block headers (with auxiliary proof of work) form mainnet.

@github-actions github-actions bot added the perf label Sep 12, 2025
@gregorydemay gregorydemay marked this pull request as ready for review September 15, 2025 09:45
@gregorydemay gregorydemay requested review from a team as code owners September 15, 2025 09:45
Copy link
Contributor

@mducroux mducroux left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for adding these useful benchmarks @gregorydemay!

Copy link
Member

@Sawchord Sawchord left a comment

Choose a reason for hiding this comment

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

Looks good, I left some questions.

Thanks @gregorydemay .

Copy link
Member

@ninegua ninegua left a comment

Choose a reason for hiding this comment

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

Thanks @gregorydemay for providing the benchmarks! They are very useful

Copy link
Member Author

@gregorydemay gregorydemay left a comment

Choose a reason for hiding this comment

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

Thanks @ninegua for the feedback!

Copy link
Contributor

@nmattia nmattia left a comment

Choose a reason for hiding this comment

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

LGTM, we can always follow up on #6740 (comment)!

@gregorydemay gregorydemay added this pull request to the merge queue Sep 26, 2025
Merged via the queue into master with commit 70b3d80 Sep 26, 2025
30 checks passed
@gregorydemay gregorydemay deleted the gdemay/XC-484-bench-add-doge-headers branch September 26, 2025 09:23
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.

6 participants