Skip to content

Conversation

mruoss
Copy link
Contributor

@mruoss mruoss commented Apr 25, 2025

TL;DR

This addresses the issue described in flyteorg/flyte#6428 where DB queries don't use the indexes on the workflows, tasks and lanuchplans table. This slows down the UI significantly and spikes the CPU usage of the database server.

Type

  • Bug Fix (Performance)
  • Feature
  • Plugin

Are all requirements met?

  • Code completed
  • Smoke tested
  • Unit tests added
  • Code documentation added
  • Any pending items have an associated Issue

Complete description

The index in the database is over the three columns (name, project, domain) - not JUST name. Therefore when filtering, we need to pass all these values in order to leverage the index.

Tracking Issue

fixes flyteorg/flyte#6428

Copy link

welcome bot commented Apr 25, 2025

Thank you for opening this pull request! 🙌

These tips will help get your PR across the finish line:

  • Most of the repos have a PR template; if not, fill it out to the best of your knowledge.
  • Sign off your commits (Reference: DCO Guide).

@mruoss mruoss force-pushed the fix-performance-issues branch from 96cbaab to f3118eb Compare April 25, 2025 13:48
@EngHabu EngHabu merged commit 794e82a into flyteorg:master May 5, 2025
10 checks passed
Copy link

welcome bot commented May 5, 2025

Congrats on merging your first pull request! 🎉

@flyte-bot
Copy link
Collaborator

🎉 This PR is included in version 1.20.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Not using DB index to list executions (slow workflows page in UI)

3 participants