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

Un-do all the reindex-algolia changes to improve memory consumption #646

Merged
merged 1 commit into from
Jul 12, 2023

Conversation

pwnage101
Copy link
Contributor

Revert c55e035 "fix: attempt to pass --dry-run correctly to task"
Revert 077f678 "fix: reindex-algolia - include UUIDs from ALL related objects"
Revert 1eb67a4 "revert: "temp: Revert reindex-algolia task changes""

ENT-7311

Description

Description of changes made

Ticket Link

Link to the associated ticket
Link title

Post-review

  • Squash commits into discrete sets of changes
  • Ensure that once the changes have been deployed to stage, prod is manually deployed

…mption

Revert c55e035 "fix: attempt to pass --dry-run correctly to task"
Revert 077f678 "fix: reindex-algolia - include UUIDs from ALL related objects"
Revert 1eb67a4 "revert: "temp: Revert reindex-algolia task changes""

ENT-7311
@pwnage101 pwnage101 force-pushed the pwnage101/ENT-7311-6-revert branch from 9cd1ae3 to 067ac71 Compare July 12, 2023 20:05
@@ -208,7 +206,7 @@ def wrapped_task(self, *args, **kwargs):
},
)
raise TaskRecentlyRunError(message)
return task(self, *args, **kwargs)
return task(self, *args, *kwargs)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note to you or me: let's follow up with a small PR to correct this to **kwargs

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh good catch

@pwnage101 pwnage101 merged commit 7886847 into master Jul 12, 2023
5 of 6 checks passed
@pwnage101 pwnage101 deleted the pwnage101/ENT-7311-6-revert branch July 12, 2023 20:49
irfanuddinahmad pushed a commit that referenced this pull request Jul 24, 2024
Un-do all the reindex-algolia changes to improve memory consumption
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