Skip to content

Add InvalidateElementsCacheJob#127

Merged
tvdeyen merged 1 commit into
mainfrom
invalidate-cache-job
May 14, 2025
Merged

Add InvalidateElementsCacheJob#127
tvdeyen merged 1 commit into
mainfrom
invalidate-cache-job

Conversation

@tvdeyen

@tvdeyen tvdeyen commented May 13, 2025

Copy link
Copy Markdown
Member

In larger applications it might be the case that one product, variant or taxon is assigned to thousands of elements via one of our three ingredients. The current implementation does not perform very well. Using a active job that can be backgrounded and has an optimized algorithm to touch all elements, their parent elements and pages should improve performance.

Comment thread app/decorators/models/alchemy/solidus/spree_variant_decorator.rb
Comment thread spec/models/spree/variant_spec.rb Outdated
@tvdeyen tvdeyen force-pushed the invalidate-cache-job branch 2 times, most recently from 65bdc01 to 86a1bb1 Compare May 13, 2025 19:32
@tvdeyen tvdeyen marked this pull request as ready for review May 13, 2025 19:32
@tvdeyen tvdeyen requested a review from mamhoff May 13, 2025 19:32
@tvdeyen tvdeyen force-pushed the invalidate-cache-job branch 2 times, most recently from d065fef to 9790f5e Compare May 13, 2025 20:05
@tvdeyen tvdeyen requested review from a team and removed request for mamhoff May 14, 2025 08:48
In larger applications it might be the case that one product, variant
or taxon is assigned to thousands of elements via one of our
three ingredients. The current implementation does not perform
very well. Using a active job that can be backgrounded and has an
optimized algorithm to touch all elements, their parent elements and
pages should improve performance.
@tvdeyen tvdeyen force-pushed the invalidate-cache-job branch from 9790f5e to 35a4ada Compare May 14, 2025 08:48
@tvdeyen tvdeyen enabled auto-merge May 14, 2025 08:56
@tvdeyen tvdeyen merged commit 9996ef2 into main May 14, 2025
37 checks passed
@tvdeyen tvdeyen deleted the invalidate-cache-job branch May 14, 2025 09:02
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