Skip to content

Improve cache hit/miss reporting for cache.get() #2418

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Harsh3006 wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Improve cache hit/miss reporting for cache.get() #2418

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
41911ad
Enhance cache panel to accurately track cache hits and misses for get…
Harsh3006 Jul 15, 2026
6aa94f9
Add tests for cache hits and misses with None and default values
Harsh3006 Jul 15, 2026
391d39c
Update pending section in changes.rst
Harsh3006 Jul 15, 2026
ce6831a
Fix cache hit/miss tracking for keyword arguments in get() method
Harsh3006 Jul 24, 2026
15f47bf
Add test for get_or_set with None to count as cache miss
Harsh3006 Aug 4, 2026
76af105
Refactor cache hit/miss logic in get() method and update related tests
Harsh3006 Aug 10, 2026
91443f5
Add cache hit/miss note to cache panel and documentation
Harsh3006 Aug 10, 2026
e134f2e
docs: clarify cache hit/miss reporting changes
Harsh3006 Aug 10, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add cache hit/miss note to cache panel and documentation
  • Loading branch information
@Harsh3006
Harsh3006 committed Aug 10, 2026
commit 91443f5826366a196359a08b82c631d18bb19963
9 changes: 8 additions & 1 deletion debug_toolbar/static/debug_toolbar/css/toolbar.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -348,7 +348,14 @@
height: 50px;
}

#djDebug .djDebugPanelTitle code {
#djDebug .djdt-cache-note {
margin: 1em 0;
padding: 8px 10px;
border: 1px solid var(--djdt-table-border-color);
background-color: var(--djdt-panel-content-table-strip-background-color);
}
#djDebug .djDebugPanelTitle code,
#djDebug .djdt-cache-note code {
display: inline;
font-size: inherit;
}
Expand Down
5 changes: 5 additions & 0 deletions debug_toolbar/templates/debug_toolbar/panels/cache.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,11 @@ <h4>{% translate "Summary" %}</h4>
</tr>
</tbody>
</table>
<p class="djdt-cache-note">
Cache hit/miss statistics for <code>cache.get()</code> calls may not always be accurate.
See <a href="https://github.com/django-commons/django-debug-toolbar/discussions/2441">the discussion</a>
for details and to share feedback about improving cache hit/miss tracking.
</p>
<h4>{% translate "Commands" %}</h4>
<table>
<thead>
Expand Down
6 changes: 6 additions & 0 deletions docs/panels.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -101,6 +101,12 @@ Cache

Cache queries. Is incompatible with Django's per-site caching.

Cache hit/miss statistics for ``cache.get()`` calls may not always be
accurate. See the `discussion`_ for details and to share feedback about
improving cache hit/miss tracking.

.. _discussion: https://github.com/django-commons/django-debug-toolbar/discussions/2441

Signals
~~~~~~~

Expand Down