-
Notifications
You must be signed in to change notification settings - Fork 122
fix: optimize principal transaction lists and etags #2370
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
base: develop
Are you sure you want to change the base?
Conversation
|
Vercel deployment URL: https://stacks-blockchain-k8yngnz6y-hirosystems.vercel.app 🚀 |
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
|
Next step is to perform smoke tests on dev to compare consistency vs prod and check query response times for accounts with large numbers of transactions |
CharlieC3
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM pending validation in dev ![]()
Introduces a new
principal_txstable to consolidate and optimize transaction tracking for principals, replacing the oldprincipal_stx_txstable with a more comprehensive solution that allows us to do direct SELECTs instead of calculating data on demand. The old table is kept intact for backwards compatibility.