Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Hotfix / Improve scores feed performance #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Hotfix / Improve scores feed performance #1

wants to merge 1 commit into from

Conversation

raulbancea
Copy link
Owner

Fixes: AACraiu#1

Changes

Fixed the n+1 problem by using eager loading instead of lazy loading

Before
Screenshot 2024-09-05 at 15 30 46

After
Screenshot 2024-09-05 at 15 31 20

@cristian-scarisoreanu
Copy link

The title should usually be Hotfix / [Issue Name] for hotfix branches and Feature / [Issue Name] for feature branches.

You can copy the name from the issue's link.

@raulbancea raulbancea changed the title n+1 fix Hotfix / Improve scores feed performance #1 Sep 9, 2024
@raulbancea raulbancea changed the title Hotfix / Improve scores feed performance #1 Hotfix / Improve scores feed performance Sep 9, 2024
@cristian-scarisoreanu cristian-scarisoreanu mentioned this pull request Sep 12, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cristian-scarisoreanu cristian-scarisoreanu cristian-scarisoreanu approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Improve scores feed performance
2 participants
@raulbancea @cristian-scarisoreanu