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

migrator: detach receivers during migration #2711

Merged
merged 2 commits into from
Sep 3, 2017
Merged

migrator: detach receivers during migration #2711

merged 2 commits into from
Sep 3, 2017

Conversation

jacquerie
Copy link
Contributor

@jacquerie jacquerie commented Sep 3, 2017

Description

The solution attempted in 16e9cb9 didn't work as that receiver was
detached by the Python interpreter executing the CLI command, not
the one executing the task code in the worker.

Fixes the performance issue caused by invenio-collections by detaching
its receiver during migration. Its work, in fact, is already handled
by inspire-dojson since inspirehep/inspire-dojson#62.

Related Issue

Closes #1640

Checklist:

  • I have all the information that I need (if not, move to RFC and look for it).
  • I linked the related issue(s) in the corresponding commit logs.
  • I wrote good commit log messages.
  • My code follows the code style of this project.
  • I've added any new docs if API/utils methods were added.
  • I have updated the existing documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

The solution attempted in 16e9cb9 didn't work as that receiver was
detached by the Python interpreter executing the CLI command, not
the one executing the task code in the worker.

Signed-off-by: Jacopo Notarstefano <[email protected]>
Fixes the performance issue caused by invenio-collections by detaching
its receiver during migration. Its work, in fact, is already handled
by inspire-dojson since inspirehep/inspire-dojson#62.

Signed-off-by: Jacopo Notarstefano <[email protected]>
@jacquerie jacquerie requested a review from jmartinm September 3, 2017 16:57
@jacquerie jacquerie merged commit 5ebda64 into inspirehep:master Sep 3, 2017
@jacquerie jacquerie deleted the detach-receivers-during-migration branch September 3, 2017 22:06
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