Skip to content
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

Add evict function for ARC #48

Closed
wants to merge 1 commit into from

Conversation

jasonsurratt
Copy link

Adds evict functionality to the ARC cache. See also #22

@hainesc
Copy link

hainesc commented Dec 6, 2019

@jasonsurratt Awesome work!

Before I create a PR, I see this one, it is almost the same with mine.

Looking forward this PR get merged as I need onEvict callback function in one of my project. I add support of onEvict callback in my fork repo.

@hainesc
Copy link

hainesc commented Dec 6, 2019

@jasonsurratt, Please know that you need to sign the CLA.

@jasonsurratt
Copy link
Author

@hainesc Thanks! I signed the CLA.

@immesys
Copy link

immesys commented Jan 21, 2020

I would really like this functionality. Any chance it can be merged soon?

@jasonsurratt
Copy link
Author

Please let me know if there is anything else I can do to get this merged. I don't have write access and it has been dangling out here a while. Thanks!

@hashicorp-cla
Copy link

hashicorp-cla commented Sep 9, 2021

CLA assistant check
All committers have signed the CLA.

@jackedelic
Copy link

Could we get this merged :) ? Need this feature to make my boss happy

@jasonsurratt
Copy link
Author

This ceased to be relevant to me about 3.5 years ago so I'll let it die. Looks like the conflicts are pretty minor but I haven't used go in so long I'm afraid I may make a silly error merging them.

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.

None yet

5 participants