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

attach eviction filling logic to set_cache #3034

Closed
wants to merge 3 commits into from

Conversation

duduyi2013
Copy link

Summary:

  • add eviction callback into cachelib
  • add eviction handling logic in cachelib wrapper and kv db tbe

Differential Revision: D61200308

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D61200308

Copy link

netlify bot commented Aug 24, 2024

Deploy Preview for pytorch-fbgemm-docs ready!

Name Link
🔨 Latest commit 08a5b4d
🔍 Latest deploy log https://app.netlify.com/sites/pytorch-fbgemm-docs/deploys/66d4c910381b8700089f4be4
😎 Deploy Preview https://deploy-preview-3034--pytorch-fbgemm-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Guanqiao Wang and others added 3 commits August 29, 2024 12:38
Summary:
X-link: pytorch/torchrec#2335

Pull Request resolved: pytorch#3031

X-link: facebookresearch/FBGEMM#129

collect performance related metrics from KV store and export them to ODS

Differential Revision: D61417980
Summary:
set cache is inserting values into L2 cache
set_async it inserting values into SSD(rocksdb)

this 2 steps could be done asynchronously in the background thread wihtout blocking the read path.
Actually, it could potentially block the next read if it take close to each train iteration time.

Differential Revision: D61418016
Summary:
X-link: facebookresearch/FBGEMM#132

- add eviction callback into cachelib
- add eviction handling logic in cachelib wrapper and kv db tbe

Differential Revision: D61200308
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D61200308

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 8346a7d.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants