Problem/Motivation

Change default backend to apcu and run cron.

Warning: apc_fetch(): apc_fetch() expects a string or array of strings. in Drupal\Core\Cache\ApcuBackend->garbageCollection() (line 247 of core/lib/Drupal/Core/Cache/ApcuBackend.php).

Proposed resolution

Remove the call to apc_fetch('').

Remaining tasks

User interface changes

API changes

CommentFileSizeAuthor
#4 2269453.patch505 bytesolli
apcu-garbagecollection.patch450 bytesolli

Comments

Anonymous’s picture

Anonymous (not verified) commented

wow, i guess we don't exercise this in tests?

also, i think it would be better to just apc_fetch() some value, so we the GC to run?

olli’s picture

olli commented

Does apcu run gc on apc_fetch()?

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

Do we need to GC at all? The next apc_fetch() will GC anyway?

In order words, I'd be fine with this patch, although we should add a corresponding comment as a replacement for the removed one.

olli’s picture

olli commented
StatusFileSize
new 2269453.patch505 bytes

Added a comment.

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs review » Reviewed & tested by the community

nice.

catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.x, thanks!

  • Commit 1f5ad10 on 8.x by catch: 
    Issue #2269453 by olli: Apc_fetch() expects a string or array of strings...

Status: Fixed » Closed (fixed)

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