Skip to content

Expose SE/DESE support to EmbeddingRocksDBWrapper for training pipeline #4227

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

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

Raahul46
Copy link

Summary:
X-link: https://github.com/facebookresearch/FBGEMM/pull/1304

Design doc: https://docs.google.com/document/d/149LdAEHOLP7ei4hwVVkAFXGa4N9uLs1J7efxfBZp3dY/edit?tab=t.0#heading=h.49t3yfaqmt54

Context:
We are enabling the usage of rocksDB checkpoint feature in KVTensorWrapper. This allows us to create checkpoints of the embedding tables in SSD. Later, these checkpoints are used by the checkpointing component to create a checkpoint and upload it it to the manifold

In this diff:

  1. We expose serialization and deserialization support through EmbeddingRocksDBWrapper
  2. Added a function to generate logs when reading through EmbeddingRocksDB/ReadOnlyEmbeddingKVDB

Differential Revision: D75489895

Copy link

netlify bot commented May 30, 2025

Deploy Preview for pytorch-fbgemm-docs ready!

Name Link
🔨 Latest commit 9adb4e9
🔍 Latest deploy log https://app.netlify.com/projects/pytorch-fbgemm-docs/deploys/6841026ffa1931000839865a
😎 Deploy Preview https://deploy-preview-4227--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 project configuration.

@facebook-github-bot
Copy link
Contributor

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

@Raahul46 Raahul46 force-pushed the export-D75489895 branch from f27f0e1 to 499bfbd Compare June 4, 2025 18:42
Raahul46 pushed a commit to Raahul46/FBGEMM-1 that referenced this pull request Jun 4, 2025
…ne (pytorch#4227)

Summary:

X-link: facebookresearch/FBGEMM#1304


Design doc: https://docs.google.com/document/d/149LdAEHOLP7ei4hwVVkAFXGa4N9uLs1J7efxfBZp3dY/edit?tab=t.0#heading=h.49t3yfaqmt54

Context:
We are enabling the usage of rocksDB checkpoint feature in KVTensorWrapper. This allows us to create checkpoints of the embedding tables in SSD. Later, these checkpoints are used by the checkpointing component to create a checkpoint and upload it it to the manifold 

In this diff:
1. We expose serialization and deserialization support through EmbeddingRocksDBWrapper
2. Added a function to generate logs when reading through EmbeddingRocksDB/ReadOnlyEmbeddingKVDB

Differential Revision: D75489895
@Raahul46 Raahul46 force-pushed the export-D75489895 branch from 499bfbd to 9764393 Compare June 4, 2025 18:44
Raahul46 pushed a commit to Raahul46/FBGEMM-1 that referenced this pull request Jun 4, 2025
…ne (pytorch#4227)

Summary:

X-link: facebookresearch/FBGEMM#1304


Design doc: https://docs.google.com/document/d/149LdAEHOLP7ei4hwVVkAFXGa4N9uLs1J7efxfBZp3dY/edit?tab=t.0#heading=h.49t3yfaqmt54

Context:
We are enabling the usage of rocksDB checkpoint feature in KVTensorWrapper. This allows us to create checkpoints of the embedding tables in SSD. Later, these checkpoints are used by the checkpointing component to create a checkpoint and upload it it to the manifold 

In this diff:
1. We expose serialization and deserialization support through EmbeddingRocksDBWrapper
2. Added a function to generate logs when reading through EmbeddingRocksDB/ReadOnlyEmbeddingKVDB

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

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

@Raahul46 Raahul46 force-pushed the export-D75489895 branch from 9764393 to 6547986 Compare June 4, 2025 18:46
Raahul46 pushed a commit to Raahul46/FBGEMM-1 that referenced this pull request Jun 4, 2025
…ne (pytorch#4227)

Summary:
Pull Request resolved: pytorch#4227

X-link: facebookresearch/FBGEMM#1304

Design doc: https://docs.google.com/document/d/149LdAEHOLP7ei4hwVVkAFXGa4N9uLs1J7efxfBZp3dY/edit?tab=t.0#heading=h.49t3yfaqmt54

Context:
We are enabling the usage of rocksDB checkpoint feature in KVTensorWrapper. This allows us to create checkpoints of the embedding tables in SSD. Later, these checkpoints are used by the checkpointing component to create a checkpoint and upload it it to the manifold

In this diff:
1. We expose serialization and deserialization support through EmbeddingRocksDBWrapper
2. Added a function to generate logs when reading through EmbeddingRocksDB/ReadOnlyEmbeddingKVDB

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

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

Raahul46 pushed a commit to Raahul46/FBGEMM-1 that referenced this pull request Jun 4, 2025
…ne (pytorch#4227)

Summary:
Pull Request resolved: pytorch#4227

X-link: facebookresearch/FBGEMM#1304

Design doc: https://docs.google.com/document/d/149LdAEHOLP7ei4hwVVkAFXGa4N9uLs1J7efxfBZp3dY/edit?tab=t.0#heading=h.49t3yfaqmt54

Context:
We are enabling the usage of rocksDB checkpoint feature in KVTensorWrapper. This allows us to create checkpoints of the embedding tables in SSD. Later, these checkpoints are used by the checkpointing component to create a checkpoint and upload it it to the manifold

In this diff:
1. We expose serialization and deserialization support through EmbeddingRocksDBWrapper
2. Added a function to generate logs when reading through EmbeddingRocksDB/ReadOnlyEmbeddingKVDB

Reviewed By: duduyi2013

Differential Revision: D75489895
@Raahul46 Raahul46 force-pushed the export-D75489895 branch from 6547986 to 28d3cc1 Compare June 4, 2025 19:05
raahul46 added 6 commits June 4, 2025 16:47
Differential Revision: D75489819
Differential Revision: D75489827
Differential Revision: D75489841
Differential Revision: D75489849
Differential Revision: D75489873
Differential Revision: D75489887
@Raahul46 Raahul46 force-pushed the export-D75489895 branch from 28d3cc1 to d5b59be Compare June 5, 2025 02:20
Raahul46 pushed a commit to Raahul46/FBGEMM-1 that referenced this pull request Jun 5, 2025
…ne (pytorch#4227)

Summary:

X-link: facebookresearch/FBGEMM#1304


Design doc: https://docs.google.com/document/d/149LdAEHOLP7ei4hwVVkAFXGa4N9uLs1J7efxfBZp3dY/edit?tab=t.0#heading=h.49t3yfaqmt54

Context:
We are enabling the usage of rocksDB checkpoint feature in KVTensorWrapper. This allows us to create checkpoints of the embedding tables in SSD. Later, these checkpoints are used by the checkpointing component to create a checkpoint and upload it it to the manifold 

In this diff:
1. We expose serialization and deserialization support through EmbeddingRocksDBWrapper
2. Added a function to generate logs when reading through EmbeddingRocksDB/ReadOnlyEmbeddingKVDB

Reviewed By: duduyi2013

Differential Revision: D75489895
@Raahul46 Raahul46 force-pushed the export-D75489895 branch from d5b59be to 0fe3a05 Compare June 5, 2025 02:22
Raahul46 pushed a commit to Raahul46/FBGEMM-1 that referenced this pull request Jun 5, 2025
…ne (pytorch#4227)

Summary:

X-link: facebookresearch/FBGEMM#1304


Design doc: https://docs.google.com/document/d/149LdAEHOLP7ei4hwVVkAFXGa4N9uLs1J7efxfBZp3dY/edit?tab=t.0#heading=h.49t3yfaqmt54

Context:
We are enabling the usage of rocksDB checkpoint feature in KVTensorWrapper. This allows us to create checkpoints of the embedding tables in SSD. Later, these checkpoints are used by the checkpointing component to create a checkpoint and upload it it to the manifold 

In this diff:
1. We expose serialization and deserialization support through EmbeddingRocksDBWrapper
2. Added a function to generate logs when reading through EmbeddingRocksDB/ReadOnlyEmbeddingKVDB

Reviewed By: duduyi2013

Differential Revision: D75489895
@Raahul46 Raahul46 force-pushed the export-D75489895 branch from 0fe3a05 to fbe232a Compare June 5, 2025 02:23
@facebook-github-bot
Copy link
Contributor

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

1 similar comment
@facebook-github-bot
Copy link
Contributor

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

Raahul46 pushed a commit to Raahul46/FBGEMM-1 that referenced this pull request Jun 5, 2025
…ne (pytorch#4227)

Summary:
Pull Request resolved: pytorch#4227

X-link: facebookresearch/FBGEMM#1304

Design doc: https://docs.google.com/document/d/149LdAEHOLP7ei4hwVVkAFXGa4N9uLs1J7efxfBZp3dY/edit?tab=t.0#heading=h.49t3yfaqmt54

Context:
We are enabling the usage of rocksDB checkpoint feature in KVTensorWrapper. This allows us to create checkpoints of the embedding tables in SSD. Later, these checkpoints are used by the checkpointing component to create a checkpoint and upload it it to the manifold

In this diff:
1. We expose serialization and deserialization support through EmbeddingRocksDBWrapper
2. Added a function to generate logs when reading through EmbeddingRocksDB/ReadOnlyEmbeddingKVDB

Reviewed By: duduyi2013

Differential Revision: D75489895
@Raahul46 Raahul46 force-pushed the export-D75489895 branch from fbe232a to 5c893ca Compare June 5, 2025 02:29
…ne (pytorch#4227)

Summary:
Pull Request resolved: pytorch#4227

X-link: facebookresearch/FBGEMM#1304

Design doc: https://docs.google.com/document/d/149LdAEHOLP7ei4hwVVkAFXGa4N9uLs1J7efxfBZp3dY/edit?tab=t.0#heading=h.49t3yfaqmt54

Context:
We are enabling the usage of rocksDB checkpoint feature in KVTensorWrapper. This allows us to create checkpoints of the embedding tables in SSD. Later, these checkpoints are used by the checkpointing component to create a checkpoint and upload it it to the manifold

In this diff:
1. We expose serialization and deserialization support through EmbeddingRocksDBWrapper
2. Added a function to generate logs when reading through EmbeddingRocksDB/ReadOnlyEmbeddingKVDB

Reviewed By: duduyi2013

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

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

@Raahul46 Raahul46 force-pushed the export-D75489895 branch from 5c893ca to 9adb4e9 Compare June 5, 2025 02:35
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