Skip to content

Commit

Permalink
Support cost-based eviction
Browse files Browse the repository at this point in the history
- Rename an internal structure TotalWeight to TotalCost.
- Lightly refactored.
  • Loading branch information
tatsuya6502 committed Aug 15, 2021
1 parent 2835b94 commit 427e67f
Showing 1 changed file with 78 additions and 85 deletions.
Loading

2 comments on commit 427e67f

@ben-manes
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FYI, cost usually means the miss penalty, so ristretto‘s use like this was awkward imho.

@tatsuya6502
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cost usually means the miss penalty

Ouch. Cost has other meaning in cache. OK. I will rename them back to weight. Thanks for letting me know!

Please sign in to comment.