Skip to content

Conversation

Kobzol
Copy link
Member

@Kobzol Kobzol commented Sep 19, 2025

Alternative to #2249.

@Kobzol Kobzol requested a review from Jamesbarford September 19, 2025 16:14
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. We can always revisit if this has bottle necks from the 5 database calls required to compose the status page proves too many.

@Kobzol
Copy link
Member Author

Kobzol commented Sep 22, 2025

Should be just four after #2252 ^^ As long as the query count doesn't scale with the number of fetched requests, as it did before, I think it'll be fine.

@Kobzol Kobzol merged commit 4e9a6bb into rust-lang:master Sep 22, 2025
11 checks passed
@Kobzol Kobzol deleted the status-page branch September 22, 2025 07:03
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