Skip to content

Improve Fetch Speed for Feed List#649

Closed
binh-dam-ibigroup wants to merge 3 commits into
dev-flexfrom
flex-fix-project-list-perf
Closed

Improve Fetch Speed for Feed List#649
binh-dam-ibigroup wants to merge 3 commits into
dev-flexfrom
flex-fix-project-list-perf

Conversation

@binh-dam-ibigroup
Copy link
Copy Markdown
Contributor

@binh-dam-ibigroup binh-dam-ibigroup commented May 13, 2026

Checklist

  • Appropriate branch selected => This PR is against the dev-flex branch
  • Any modified or new methods or classes have helpful JavaDoc and code is thoroughly commented
  • The description lists all applicable issues this PR seeks to resolve
  • [na] The description lists any configuration setting(s) that differ from the default settings
  • All tests and CI builds passing

Description

This PR improves the loading speed of the list of feeds in a project. This is done by adding a filter by feedSourceId to a lookup that inherently applies to a specific feed source.

Testing the JS query in Mongo yields 100x speed improvement when dealing with a large number (~200k) of feed versions.

@binh-dam-ibigroup binh-dam-ibigroup requested a review from br648 May 13, 2026 13:57
Copy link
Copy Markdown
Contributor

@br648 br648 left a comment

Choose a reason for hiding this comment

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

Should this be merged into dev and not dev-flex? All good otherwise.

@binh-dam-ibigroup
Copy link
Copy Markdown
Contributor Author

Should this be merged into dev and not dev-flex? All good otherwise.

This PR is intended for dev-flex. I will do another PR for the dev branch too.

@binh-dam-ibigroup
Copy link
Copy Markdown
Contributor Author

@br648 Actually I'm gonna close this in favor of #650 against dev (fewer reviews).

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