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

Change the Translated pattern cron task to run many smaller tasks #620

Merged
merged 4 commits into from
Feb 1, 2024

Conversation

dd32
Copy link
Member

@dd32 dd32 commented Feb 1, 2024

Recently the cron task 'pattern_import_translations_to_directory' has been running into memory limits.

While this shouldn't happen, it makes sense to just split this task up into many smaller jobs.

@dd32 dd32 merged commit a84a75b into trunk Feb 1, 2024
2 of 3 checks passed
@dd32 dd32 deleted the fix/cron-memory-exhaustion branch February 1, 2024 06:19
@pkevan
Copy link
Contributor

pkevan commented Feb 1, 2024

seems this change caused some fatal errors, which I thought were due to glotpress not being activated, but doing so caused the crons to fail.

@akirk committed the following to hopefully fix it: WordPress/wordpress.org@78e808b, and the plugin is now not activated on the site.

dd32 added a commit that referenced this pull request Feb 23, 2024
This is the root cause that caused #620, and is now tracked upstream as WordPress/gutenberg#59300
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