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

Optimize federated language updates to avoid unnecessary db writes #2786

Merged
merged 4 commits into from
Mar 30, 2023

Conversation

Nutomic
Copy link
Member

@Nutomic Nutomic commented Mar 21, 2023

Change the database methods for languages so that they dont perform any writes if the language remains unchanged. This avoids unnecessary db writes from federation code.

@Nutomic Nutomic requested a review from dessalines as a code owner March 21, 2023 20:12
@Nutomic Nutomic force-pushed the optimize-language-updates branch from 65001ac to cbe41b1 Compare March 27, 2023 20:34
@Nutomic
Copy link
Member Author

Nutomic commented Mar 27, 2023

Updated, tests should be passing now.

@dessalines
Copy link
Member

Site langs still failing.

@Nutomic
Copy link
Member Author

Nutomic commented Mar 28, 2023

Fixed

@dessalines dessalines merged commit 165b19e into main Mar 30, 2023
dessalines added a commit that referenced this pull request Apr 26, 2023
…2786)

* Optimize federated language updates to avoid unnecessary db writes (fixes #2772)

* fix tests

* fix test, rename functions

---------

Co-authored-by: Dessalines <[email protected]>
Nutomic added a commit that referenced this pull request Apr 27, 2023
…2786)

* Optimize federated language updates to avoid unnecessary db writes (fixes #2772)

* fix tests

* fix test, rename functions

---------

Co-authored-by: Dessalines <[email protected]>
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