...instead of huge flat directories, that is.

I was fearing that huge directories would have a negative effect on performance. This is not true, according to ogi in #1211166-16: Create separate directories for each cache bins. But now I already creted the code for this... so I submit it for review in a separate issue.

In my patch, set_encoding() sets various defaults for entity_cache_X bins, which may not be useful given the above, but I'm leaving it like this for now. The new directory level ogi mentions in #1211166-16: Create separate directories for each cache bins is possible with set_encoding(1).

This patch depends on #1211166-18: Create separate directories for each cache bins and #1488870-2: Reorganize cache expiry behavior being applied already. Those issues are way more important than this one.

CommentFileSizeAuthor
filecache-trees.patch16.02 KBroderik

Comments

roderik’s picture

By the way, the patch absolutely depends on #1211166-18: Create separate directories for each cache bins.

But if you really want to use it without applying #1488870-2: Reorganize cache expiry behavior first: edit the patch before applying: in the last hunk, replace the two occurrences of '$calc_size' by '$gather_stats'.

ogi’s picture

Assigned: Unassigned » ogi
Issue summary: View changes
Status: Needs review » Active
pfrenssen’s picture

Status: Active » Closed (outdated)

Drupal 7 is end-of-life, closing.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.