Person:+delete timeouts : Person merging needs to be done asynchronously
Bug #162510 reported by
Christian Reis
This bug affects 5 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Fix Released
|
Critical
|
Gavin Panella |
Bug Description
The Person merge code is robust but will timeout for records with lots of references to update.
The best way of addressing this is to queue merges to be done asynchronously.
This should be done for both user requested and admin merges.
See bug 728471 about the UI work needed to solve this issue.
Related branches
lp://staging/~allenap/launchpad/async-person-merge-162510
- Gavin Panella (community): Approve
- Данило Шеган (community): Approve
-
Diff: 865 lines (+527/-44)16 files modifieddatabase/schema/security.cfg (+124/-0)
lib/canonical/config/schema-lazr.conf (+7/-1)
lib/canonical/launchpad/browser/logintoken.py (+11/-0)
lib/canonical/launchpad/scripts/logger.py (+10/-0)
lib/lp/registry/browser/peoplemerge.py (+20/-2)
lib/lp/registry/configure.zcml (+21/-13)
lib/lp/registry/enum.py (+6/-0)
lib/lp/registry/interfaces/person.py (+15/-0)
lib/lp/registry/interfaces/persontransferjob.py (+36/-0)
lib/lp/registry/model/person.py (+12/-0)
lib/lp/registry/model/persontransferjob.py (+85/-13)
lib/lp/registry/tests/test_membership_notification_job.py (+20/-4)
lib/lp/registry/tests/test_person.py (+24/-0)
lib/lp/registry/tests/test_person_merge_job.py (+123/-0)
lib/lp/registry/tests/test_process_job_sources_cronjob.py (+4/-8)
lib/lp/services/job/model/job.py (+9/-3)
Changed in launchpad-registry: | |
importance: | Undecided → Low |
status: | New → Triaged |
description: | updated |
summary: |
- Merging people times out updating POSubmission and POMsgSet + Person merging needs to be done asynchronously |
description: | updated |
tags: |
added: featire tech-debt removed: registry-people |
tags: |
added: feature removed: featire |
Changed in launchpad-registry: | |
milestone: | none → series-3.1 |
tags: | added: merge-deactivate |
tags: | added: canonical-losa-lp |
summary: |
- Person merging needs to be done asynchronously + Person:+delete timeouts : Person merging needs to be done asynchronously |
tags: | added: pg83 |
tags: | removed: pg83 |
Changed in launchpad: | |
importance: | High → Critical |
Changed in launchpad: | |
assignee: | nobody → Gavin Panella (allenap) |
Changed in launchpad: | |
status: | Triaged → In Progress |
description: | updated |
Changed in launchpad: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
OOPS-759D2129 might be related. POSubmission and POMsgSet tables are gone since the Translations DB refactoring, and TranslationMessage might have inherited the problem.