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

Translation coverage produces a lot of sql queries #2139

Closed
timobrembeck opened this issue Mar 22, 2023 · 0 comments · Fixed by #2140
Closed

Translation coverage produces a lot of sql queries #2139

timobrembeck opened this issue Mar 22, 2023 · 0 comments · Fixed by #2140
Assignees
@timobrembeck
Labels
🐛 bug Something isn't working ❗ prio: medium Should be scheduled in the forseeable future. ☺️ effort: low Should be doable in <4h
Milestone
23Q1

Comments

@timobrembeck
Copy link
Member

Describe the Bug

We got the translation coverage down to a constant number of queries some time ago, but now the view has become very slow again, so apparently there was a side effect on one of our latest changes.

Steps to Reproduce

  1. Go to translation coverage
  2. See number of queries in Django debug toolbar

Expected Behavior

The number of queries should not depend on the number of pages in the database

Actual Behavior

There are some queries executed in a loop, so the more pages, the more sql queries
@timobrembeck timobrembeck added 🐛 bug Something isn't working ❗ prio: medium Should be scheduled in the forseeable future. ☺️ effort: low Should be doable in <4h labels Mar 22, 2023
@timobrembeck timobrembeck added this to the 23Q1 milestone Mar 22, 2023
@timobrembeck timobrembeck self-assigned this Mar 22, 2023
@timobrembeck timobrembeck mentioned this issue Mar 22, 2023
Merged
This was referenced Mar 23, 2023
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@timobrembeck timobrembeck

Labels
🐛 bug Something isn't working ❗ prio: medium Should be scheduled in the forseeable future. ☺️ effort: low Should be doable in <4h
Projects
None yet
Milestone
23Q1
Development

Successfully merging a pull request may close this issue.

Improve performance of translation coverage digitalfabrik/integreat-cms
1 participant
@timobrembeck