-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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.
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
Optimize the content of the list of archived workflows sent to front-end #12030
Comments
We do some server-side filtering on the
Since it is the same API used by everything else, this feature is for more server-side filtering that the UI could use? |
Correct |
So I didn't think there was much room to improve this (but I haven't looked too closely), but I did discover a possible performance regression with this on the Archived Workflow side: #11121 (comment) -- all fields are getting |
Not needed anymore due to improvements in #13021 |
Summary
Currently, we retrieve the list of both live workflows and archived workflows where each workflow contains the entire workflow object, including spec, status, etc. Some of this information may not be needed for the workflow list view (they are only required for workflow details page), but they take up a lot of memory, which might overload the UI.
We should only return the list of workflows with the required information from each workflow object.
Related #12025
Message from the maintainers:
Love this enhancement proposal? Give it a 👍. We prioritise the proposals with the most 👍.
The text was updated successfully, but these errors were encountered: