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

Simplify subscription processing logic #170

Closed
aldesantis opened this issue Nov 19, 2020 · 1 comment
Closed

Simplify subscription processing logic #170

aldesantis opened this issue Nov 19, 2020 · 1 comment
Assignees
Labels
enhancement Improves an existing feature.

Comments

@aldesantis
Copy link
Member

Currently, SolidusSubscriptions::Processor is a bit too clever in how it tries to group subscriptions/installments together. As a result, it's very complicated and it generates background jobs that doo too much stuff, trying to process multiple installments at the same time.

We should streamline both the processor and the background job that fulfills subscriptions by distributing the workload horizontally as much as possible, which should also improve performance.

@aldesantis aldesantis self-assigned this Jan 15, 2021
@aldesantis aldesantis added the enhancement Improves an existing feature. label Jan 15, 2021
@ChristianRimondi
Copy link
Contributor

Closed by #172

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improves an existing feature.
Projects
None yet
Development

No branches or pull requests

2 participants