Skip to content

Conversation

@FriedCosey
Copy link

Summary:
See https://fb.workplace.com/groups/1404957374198553/permalink/1610214197006202/

Similar to D85999577, we store the hash_zch_runtime_meta when it is being look up in mc_modules.py in raw_id_tracker then access it in batched_embedding_kernel.py which will then be streamed to the inference side (see the other diffs in this stack D87810125)

Differential Revision: D88623165

Joey Yang added 2 commits December 8, 2025 13:29
…3598)

Summary:

See https://fb.workplace.com/groups/1404957374198553/permalink/1610214197006202/

This diff extends `raw_id_tracker` to store `hash_zch_runtime_meta` which will be alongside with `hash_zch_identities` when presented.

Note that it is possible that a mpzch table only has `hash_zch_identities` without `hash_zch_runtime_meta` but is not true vice versa.

Differential Revision: D88600497
Summary:
See https://fb.workplace.com/groups/1404957374198553/permalink/1610214197006202/

Similar to D85999577, we store the `hash_zch_runtime_meta` when it is being look up in `mc_modules.py` in `raw_id_tracker` then access it in `batched_embedding_kernel.py` which will then be streamed to the inference side (see the other diffs in this stack D87810125)

Differential Revision: D88623165
@meta-codesync
Copy link
Contributor

meta-codesync bot commented Dec 8, 2025

@FriedCosey has exported this pull request. If you are a Meta employee, you can view the originating Diff in D88623165.

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Dec 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported meta-exported

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant