Skip to content

UPDATED: Disable calendar li_html_cache object cache entirely because it's buggy and the solution to performance problems isn't object caching#828

Merged
GaryJones merged 1 commit intoAutomattic:developfrom
jerclarke:fix-disable-calendar-per-user-object-cache
Dec 9, 2025
Merged

UPDATED: Disable calendar li_html_cache object cache entirely because it's buggy and the solution to performance problems isn't object caching#828
GaryJones merged 1 commit intoAutomattic:developfrom
jerclarke:fix-disable-calendar-per-user-object-cache

Conversation

@jerclarke
Copy link
Copy Markdown
Contributor

Re: #505

RE-DONE TO AVOID MERGE HELL

Previous discussion: #755

My analysis of this issue points to bugs in what is being loaded in the HTML, rather than database queries, as the cause of slowness in the calendar, so there's no reason to "save" the object cache which is currently totally broken (it is never invalidated, so out-of-date post info just hangs around until the cache clears itself eventually).

We should remove this cache ASAP so that the calendar is accurate, and deal with performance problems separately.

…gy and the solution to performance problems isn't object caching

Re: Automattic#505

RE-DONE TO AVOID MERGE HELL

Previous discussion: Automattic#755

My analysis of this issue points to bugs in what is being loaded in the HTML, rather than database queries, as the cause of slowness in the calendar, so there's no reason to "save" the object cache which is currently totally broken (it is never invalidated, so out-of-date post info just hangs around until the cache clears itself eventually).

We should remove this cache ASAP so that the calendar is accurate, and deal with performance problems separately.
@GaryJones GaryJones merged commit fd4f59b into Automattic:develop Dec 9, 2025
5 checks passed
@GaryJones GaryJones added this to the Next (minor) milestone Dec 12, 2025
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