Skip to content

Conversation

@bitfaster
Copy link
Owner

No description provided.

int victimFreq = this.cmSketch.EstimateFrequency(victimKey);
int candidateFreq = this.cmSketch.EstimateFrequency(candidateKey);

// TODO: random factor when candidate freq < 5

Check notice

Code scanning / devskim

A "TODO" or similar was left in source code, possibly indicating incomplete functionality

Suspicious comment
itemCount = Math.Min(itemCount, this.Count);
var candidates = new List<LfuNode<K, V>>(itemCount);

// TODO: this is LRU order eviction, Caffeine is based on frequency

Check notice

Code scanning / devskim

A "TODO" or similar was left in source code, possibly indicating incomplete functionality

Suspicious comment
@bitfaster bitfaster merged commit a298f4a into main Dec 3, 2023
@coveralls
Copy link

Coverage Status

coverage: 99.275% (+0.04%) from 99.236%
when pulling baa3919 on users/alexpeck/renamecore
into e3cddc7 on main.

@bitfaster bitfaster deleted the users/alexpeck/renamecore branch December 3, 2023 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants