Skip to content

events: generate schemas for metrics #8052

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.

Sign up for GitHub

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

Jump to bottom
Open
mmcgee-jump wants to merge 1 commit into
base: main
Choose a base branch
from
Open

events: generate schemas for metrics #8052

mmcgee-jump wants to merge 1 commit into from

Conversation

@mmcgee-jump
Copy link
Contributor

@mmcgee-jump mmcgee-jump commented Jan 27, 2026 •
edited
Loading

Blocked on metric coherence / rename

@mmcgee-jump mmcgee-jump added this to the v1.0 milestone Jan 27, 2026
@github-actions
Copy link

github-actions bot commented Jan 27, 2026

Performance Measurements ⏳

Suite Baseline New Change
backtest mainnet-368528500-perf per slot 0.069821 s 0.069827 s 0.009% ✅
backtest mainnet-368528500-perf snapshot load NaN s NaN s NaN% ✅
backtest mainnet-368528500-perf total elapsed 69.821145 s 69.826553 s 0.008% ✅
firedancer mem usage with mainnet.toml NaN GiB NaN GiB NaN% ✅

ripatel-fd
ripatel-fd reviewed Jan 29, 2026
View reviewed changes
src/disco/events/schema/metrics_accdb.json
Copy link
Contributor

@ripatel-fd ripatel-fd Jan 29, 2026

Choose a reason for hiding this comment

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

Can we format these with a generic JSON formatter? This is hard to work with

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ripatel-fd ripatel-fd ripatel-fd left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

v1.0

Development

Successfully merging this pull request may close these issues.

3 participants

@mmcgee-jump @ripatel-fd