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

Nobids/improve notification speed #2691

Merged
merged 12 commits into from
Nov 14, 2023
Merged

Conversation

peterbitfly
Copy link
Collaborator

@peterbitfly peterbitfly commented Nov 14, 2023

🤖 Generated by Copilot at 31375fc

This pull request improves the notification service for validator attestations by using epoch-based queries and PostgreSQL as the data source. It also defines new types for Epoch and ValidatorIndex, and adds a new function GetValidatorAttestationHistoryForNotifications to the db package.

db/db.go Show resolved Hide resolved
db/db.go Show resolved Hide resolved
services/notifications.go Outdated Show resolved Hide resolved
services/notifications.go Outdated Show resolved Hide resolved
services/notifications.go Show resolved Hide resolved
services/notifications.go Outdated Show resolved Hide resolved
services/notifications.go Show resolved Hide resolved
db/db.go Outdated Show resolved Hide resolved
Copy link
Contributor

@D13ce D13ce left a comment

Choose a reason for hiding this comment

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

lgtm

@peterbitfly peterbitfly merged commit 4c3a400 into master Nov 14, 2023
3 checks passed
@recy21 recy21 deleted the NOBIDS/improve_notification_speed branch December 4, 2023 20:49
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.

2 participants