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 the two-choice random cache eviction policy #16828

Merged

Conversation

ChunxuTang
Copy link
Member

What changes are proposed in this pull request?

The PR adds the two-choice random cache eviction policy. The algorithm selects two random page IDs and evicts the one least recently used.

Why are the changes needed?

From some evaluation (https://danluu.com/2choices-eviction/), the two-choice random policy has competitive performance compared with LRU.

Does this PR introduce any user facing changes?

Users can configure the new cache eviction policy.

@alluxio-bot
Copy link
Contributor

Thank you for your pull request.
In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement (CLA).
It's all electronic and will take just a few minutes. Please download CLA form here, sign, and e-mail back to [email protected]

@ChunxuTang
Copy link
Member Author

CLA signed and sent back to [email protected].

Copy link
Contributor

@beinan beinan left a comment

Choose a reason for hiding this comment

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

lgtm

@beinan
Copy link
Contributor

beinan commented Feb 1, 2023

alluxio-bot, merge this please.

@alluxio-bot alluxio-bot merged commit 530e720 into Alluxio:master Feb 1, 2023
@Xenorith Xenorith added the area-client Alluxio client label Feb 28, 2023
jiacheliu3 pushed a commit to jiacheliu3/alluxio that referenced this pull request May 16, 2023
…cache eviction policy

The PR adds the two-choice random cache eviction policy. The algorithm
selects two random page IDs and evicts the one least recently used.

From some evaluation (https://danluu.com/2choices-eviction/), the
two-choice random policy has competitive performance compared with LRU.

Users can configure the new cache eviction policy.

pr-link: Alluxio#16828
change-id: cid-f9a140208b3a6938ddc70766abfb3d9f90c722f0
jiacheliu3 pushed a commit to jiacheliu3/alluxio that referenced this pull request May 16, 2023
…cache eviction policy

The PR adds the two-choice random cache eviction policy. The algorithm
selects two random page IDs and evicts the one least recently used.

From some evaluation (https://danluu.com/2choices-eviction/), the
two-choice random policy has competitive performance compared with LRU.

Users can configure the new cache eviction policy.

pr-link: Alluxio#16828
change-id: cid-f9a140208b3a6938ddc70766abfb3d9f90c722f0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-client Alluxio client
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants