JanusGraph / JanusGraph/janusgraph

Emit stats for cache hits in addition to misses, retrievals, etc in ExpirationKCVSCache

Open
#267 0 comments 0 reactions 0 assignees View on GitHub
kind/enhancement
Dominant language
Java
Stars
5.8k
Forks
1.2k
Avg merge
13h 53m
Merged PRs (30d)
6

Description

We currently only emit stats for retrievals, misses, expirations, however we do not emit stats for hits explicitly. Without hits, displaying the cache hit ratio is very awkward in systems like graphite. Also, it's easier to reason about hit ratio than miss ratio.

Contributor guide

Open the contributing guide

Research direction

Start by locating ExpirationKCVSCache and tracing where retrieval, miss, and expiration statistics are emitted. Add an explicit hit statistic alongside those existing events so cache hit ratios can be displayed directly. Done means cache hits are reported consistently with the other cache statistics.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
databases, observability
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.