-
Notifications
You must be signed in to change notification settings - Fork 8
Residency #101
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.
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
base: main
Are you sure you want to change the base?
Residency #101
Conversation
b33d9b7 to
ac72151
Compare
20e4734 to
b78e765
Compare
|
Note: this PR adds a dependency on Boost |
|
Another option that we could use to support both |
f2760f0 to
2e86d4f
Compare
|
|
Just a note here. This PR implements a caching algorithm, but currently is unconnected with the rest of DYAD. We will use this algorithm to implement consumer-side file-based caching (in a separate PR). |
wangvsa
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. I will merge this PR first then integrate it with my side of code to implement the consumer-side caching. The integration may take some time and testing will be time-consuming as well.
This PR depends on PR #91
This PR implements set-associative cache with LRU replacement scheme as well as basic priority-based replacement scheme.
This includes a test.