The old statistics didn't cover every operation (flush and delete weren't reported), and it was difficult to see exactly what the sequence of events was. This patch improves the reporting by making it tabular, by recording every event, in the order that it happens, and reporting whether a get action results in a hit or not. It also differentiates between get and getMulti. In fact, in my testing with the new statistics, it appears that getMulti is always failing (different issue).

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

robertDouglass’s picture

Version: 6.x-1.x-dev » 7.x-1.x-dev
Status: Needs review » Patch (to be ported)

#911226 by robertDouglass: Added Improved statistics reporting.
committed to 6.x.

Jeremy’s picture

I suspect the multi-get that's "failing" is simple the wildcard support -- it makes a lot of queries, and most will not match anything. (We're working on a patch to minimize the number of queries required in 7.x, and will backport it to 6.x when it's complete) In any case, I'm eager to see this ported to 7.x too, proper reporting like this is quite useful. Thanks, Robert!

robertDouglass’s picture

Status: Patch (to be ported) » Fixed
FileSize
6.34 KB

Here's the D7 patch ... which I'm committing.

Status: Fixed » Closed (fixed)

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