Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: remove accidental cartesian product of large tables in pending proposal date query #62

Merged
merged 1 commit into from
Nov 25, 2024

Conversation

zone117x
Copy link
Member

This simple query was taking dozens of seconds after the block_pushes table started getting populated. This optimization brings that down to ~300ms

Copy link

Vercel deployment URL: https://signer-metrics-ka1k8ch4p-hirosystems.vercel.app 🚀

@zone117x zone117x merged commit 1bcc4c1 into main Nov 25, 2024
4 checks passed
@zone117x zone117x deleted the fix/signer-prom-metrics-perf branch November 25, 2024 16:26
blockstack-devops pushed a commit that referenced this pull request Nov 25, 2024
## [0.12.1](v0.12.0...v0.12.1) (2024-11-25)

### Bug Fixes

* remove accidental cartesian product of `blocks_pushes` * `blocks` in pending proposal date query ([#62](#62)) ([1bcc4c1](1bcc4c1))
@blockstack-devops
Copy link

🎉 This PR is included in version 0.12.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Copy link

codecov bot commented Nov 25, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants