On rule save we have currently a cache_clear_all which is way too much and is a PITA when developing.

We should only invalidate what is needed.

See \Drupal\color\EventSubscriber\ColorConfigCacheInvalidator::onChange

      $this->cacheTagsInvalidator->invalidateTags(['library_info']);

Comments

axel.rutz created an issue. See original summary.

pookmish’s picture

pookmish commented

Thanks for finding that example. I was looking for something like that during the initial build. I'll try to incorporate soon.

  • pookmish committed 650fba6 on 8.x-1.x 
    Issue #2869551: On rule save clear only needed caches
pookmish’s picture

pookmish commented
Status: Active » Needs review
pookmish’s picture

pookmish commented

That should help speed things up alot.

pookmish’s picture

pookmish commented
Status: Needs review » Fixed

Added services so that it only rebuilds the file and invalidates cache when an asset injector is saved/deleted.

  • pookmish committed 650fba6 on 8.x-2.x 
    Issue #2869551: On rule save clear only needed caches

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.