Skip to content

Conversation

Jamesbarford
Copy link
Contributor

  • Rework SQL queries, for some reason there was a query there that we weren't using so I've deleted that. I also bring back parent requests along with their jobs. However we only parse the jobs if the parent_active flag is set. Which will save a bit of time.
  • Delete all gnarly frontend transformations and push them to the server side. The remaining transformations are UI specific; namely drawing ---- under the requests that are in progress
  • Sort the jobs for the collectors (in_progress, queued, completed)
  • Removed request from collectors
  • Add asterisk for a parent request which is completed but still needs benchmarking due to backfilling.

The ui now looks like this;
image

I've not added a filter for filtering in/out completed. I'd prefer to do that in another PR as that involves interactivity which I've not looked at how to do.

@Jamesbarford Jamesbarford marked this pull request as draft August 28, 2025 14:32
@Jamesbarford Jamesbarford force-pushed the feat/status-page-ui-pt2 branch from f002589 to e0f25f5 Compare August 28, 2025 14:40
@Jamesbarford Jamesbarford marked this pull request as ready for review August 28, 2025 14:40
Copy link
Member

@Kobzol Kobzol left a comment

Choose a reason for hiding this comment

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

Thanks! I'll merge this now, but after I examined the code in more detail, I think that the DB/backend side has ballooned in complexity a bit, I'll try to refactor it and send a PR for a review.

@Kobzol Kobzol merged commit fa0510c into rust-lang:master Aug 29, 2025
11 checks passed
@Jamesbarford Jamesbarford deleted the feat/status-page-ui-pt2 branch September 3, 2025 12:46
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