Skip to content

Add metadata column family fields to EmbeddingRocksDB (#5890)#5890

Open
lizhe-ji wants to merge 2 commits into
pytorch:mainfrom
lizhe-ji:export-D108357928
Open

Add metadata column family fields to EmbeddingRocksDB (#5890)#5890
lizhe-ji wants to merge 2 commits into
pytorch:mainfrom
lizhe-ji:export-D108357928

Conversation

@lizhe-ji

@lizhe-ji lizhe-ji commented Jun 12, 2026

Copy link
Copy Markdown

Summary:

Declare the metadata column family member fields (metadata_cf_handles_,
cf_options_) and add the required includes. These are used by the metadata
methods and existing-method wiring that follow in the stack.

Differential Revision: D108357928

Lizhe Ji added 2 commits June 15, 2026 11:03
…pytorch#5883)

Summary:
X-link: facebookresearch/FBGEMM#2802


Declare two new EmbeddingRocksDB member fields used by the metadata column
family feature later in the stack:
- metadata_dim_: width of the metaheader stored in the separate metadata CF
- total_rows_written_: cumulative count of rows written

Reviewed By: EddyLXJ

Differential Revision: D108342270
Summary:

Declare the metadata column family member fields (metadata_cf_handles_,
cf_options_) and add the required includes. These are used by the metadata
methods and existing-method wiring that follow in the stack.

Differential Revision: D108357928
@meta-codesync meta-codesync Bot changed the title Add metadata column family fields to EmbeddingRocksDB Add metadata column family fields to EmbeddingRocksDB (#5890) Jun 15, 2026
@lizhe-ji lizhe-ji force-pushed the export-D108357928 branch from d9053a6 to 581a46b Compare June 15, 2026 18:04
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.

1 participant