add metrics for vinyl.cache
Open
Nobody has claimed this yet.
Stale
- Dominant language
- Lua
- Stars
- 40
- Forks
- 28
- Avg merge
- 15h 3m
- Merged PRs (30d)
- 4
Description
It is currently difficult to evaluate the efficiency of tuple_cache and whether increasing it makes sense.
Let's add some new metrics for vinyl cache:
- eviction count (index:stat().cache.evict)
- cache hit (index:stat().cache.get)
- cache miss (index:stat().get - index:stat().cache.get)
Also there might be some another useful metrics in index:stat().
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the vinyl cache metric collection and the index:stat() call in this Lua metrics library. Compare index:stat().cache.evict and index:stat().cache.get with index:stat().get, then add metrics for evictions, hits, and misses; review the remaining index:stat() fields for any clearly useful additions and verify the collected values with the project's existing checks.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- lua
- Domain
- observability
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100