Skip to content

Add known targets to the /info endpoint and optimize it#2384

Merged
Kobzol merged 4 commits intorust-lang:masterfrom
Kobzol:metrics
Feb 6, 2026
Merged

Add known targets to the /info endpoint and optimize it#2384
Kobzol merged 4 commits intorust-lang:masterfrom
Kobzol:metrics

Conversation

@Kobzol
Copy link
Member

@Kobzol Kobzol commented Feb 6, 2026

This should allow us to remove the hardcoded set of targets in the UI, to make it easier to work with multiple targets. It should also make the /info endpoint (and thus the compare and graphs pages) faster to load.

It has to iterate through all known commits, for very little value - with the status page, we can see the history of recent benchmark runs, and I doubt that people scrolled to the bottom of the compare/graphs page to check the updated at date anyway.
To make the `/info` endpoint faster.
@Kobzol Kobzol requested a review from Jamesbarford February 6, 2026 10:29
Copy link
Contributor

@Jamesbarford Jamesbarford left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@Kobzol Kobzol added this pull request to the merge queue Feb 6, 2026
Merged via the queue into rust-lang:master with commit c45fd30 Feb 6, 2026
14 checks passed
@Kobzol Kobzol deleted the metrics branch February 6, 2026 11:11
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