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.

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

move registry refresh out of the observer loop #3199

Merged
merged 1 commit into from
Apr 19, 2023

Conversation

wfjsw
Copy link
Contributor

@wfjsw wfjsw commented Apr 19, 2023

This resolves the $O(n^2)$ problem in the observer and vastly boosts the initialization performance.

I have noticed no drawbacks yet.

@JanCizmar JanCizmar requested a review from stepan662 April 19, 2023 13:35
@stepan662
Copy link
Collaborator

Thanks @wfjsw :)

@stepan662 stepan662 merged commit 243da10 into tolgee:main Apr 19, 2023
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