Skip to content

Reduce CPU usage loading large numbers of items #1768

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
Merged
merged 1 commit into from
May 8, 2022
Merged

Reduce CPU usage loading large numbers of items #1768

merged 1 commit into from
May 8, 2022

Conversation

huntharo
Copy link
Contributor

@huntharo huntharo commented May 8, 2022

Fixes #1767

Checklist

  • only relevant code is changed (make a diff before you submit the PR)
  • run tests npm run test
  • tests are included

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.1%) to 90.554% when pulling 5ba3b15 on huntharo:backend-load-performance into a0daee7 on i18next:master.

@huntharo huntharo mentioned this pull request May 8, 2022
Closed
@adrai
Copy link
Member

adrai commented May 8, 2022

lgtm

1 similar comment
@jamuhl
Copy link
Member

jamuhl commented May 8, 2022

lgtm

@adrai adrai merged commit 448e2be into i18next:master May 8, 2022
@adrai
Copy link
Member

adrai commented May 8, 2022

released with v21.8.0

@adrai adrai mentioned this pull request May 11, 2022
Closed
adrai added a commit that referenced this pull request May 11, 2022
@adrai
fix regression introduced with #1768, fixes #1772 and #1773
bd2fa9a
@huntharo huntharo deleted the backend-load-performance branch May 25, 2022 21:24
@adrai adrai mentioned this pull request Jun 3, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Large numbers of namespaces * languages use higher CPU than necessary - O(n^2)
4 participants
@huntharo @coveralls @adrai @jamuhl