The benefit gained by statically caching the views bookmark counts can far outweigh the the 1K of memory it takes to save the result of a SQL query on the count. Since this is a "public" API, it's unknown if the user will execute it multiple times in one request. Let's cache the results to avoid duplicate SQL queries.

CommentFileSizeAuthor
views_bookmark_cache.patch1.68 KBquicksketch

Comments

quicksketch’s picture

Status: Needs review » Fixed

Committed.

Anonymous’s picture

Status: Fixed » Closed (fixed)