Attached patch removes getCacheableMetadata() method which was adding an 'extensions' tag to all config objects. I'm nots sure the purpose of this tag. It doesn't seem that Drupal core uses it.

This method led us to be invalidating our dynamic cache very often, because Acquia Connector was getting this config override and clearing frequently in its cron runs.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

moshe weitzman created an issue. See original summary.

moshe weitzman’s picture

Title: Stop setting 'extensions' cache tag on unrelated config obejcts » Stop adding 'extensions' cache tag on unrelated config obejcts
moshe weitzman’s picture

Title: Stop adding 'extensions' cache tag on unrelated config obejcts » Stop adding 'extensions' cache tag on unrelated config objects

Status: Needs review » Needs work

The last submitted patch, no-tag.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

moshe weitzman’s picture

Priority: Normal » Critical
Status: Needs work » Needs review
FileSize
646 bytes

Woops - honor the interface.

moshe weitzman’s picture

  • rlhawk committed 750f7ef on 8.x-1.x authored by moshe weitzman
    Issue #3080971 by moshe weitzman: Stop adding 'extensions' cache tag on...
rlhawk’s picture

Status: Needs review » Fixed

Looks good, thanks. I simplified it in the way that Drupal core does to just:

return new CacheableMetadata();

Status: Fixed » Closed (fixed)

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