Skip to content

[v1.13] Add embbedings database metrics to stats #820

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.

Sign up for GitHub

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

Jump to bottom
Open
2 tasks
Strift opened this issue Feb 20, 2025 · 0 comments
Open
2 tasks

[v1.13] Add embbedings database metrics to stats #820

Strift opened this issue Feb 20, 2025 · 0 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@Strift
Copy link
Contributor

Strift commented Feb 20, 2025

Following the changes related to Meilisearch v1.13.0 mega issue:

Description

Update the stats object to include the new embbedings database metrics.

For further reference, see the related Meilisearch engine issue.

Specifications

  • Modify the stats API to include indexes.INDEXNAME.numberOfEmbeddedDocuments in SDK responses
  • Modify the stats API to include indexes.INDEXNAME.numberOfEmbeddings in SDK responses
@Strift Strift added enhancement New feature or request good first issue Good for newcomers labels Feb 20, 2025
@pratts pratts mentioned this issue Mar 31, 2025
Merged
3 tasks
meili-bors bot added a commit that referenced this issue Apr 3, 2025
@meili-bors @pratts
Merge #830
385f132 
830: Adds usedDatabaseSize to stats, document and embeddings database metrics to index stats r=curquiza a=pratts

# Pull Request

## Related issue
Fixes #819 #820 #821 

## What does this PR do?
- Adds `usedDatabaseSize` to stats
- Adds documents database metrics to stats
- Adds embbedings database metrics to stats

## PR checklist
Please check if your PR fulfills the following requirements:
- [x] Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
- [x] Have you read the contributing guidelines?
- [x] Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!


Co-authored-by: Prateek Sharma <[email protected]>
meili-bors bot added a commit that referenced this issue Apr 4, 2025
@meili-bors @pratts @prateek-ec
Merge #830
cfc07ea 
830: Add usedDatabaseSize to stats, document and embeddings database metrics to index stats r=curquiza a=pratts

# Pull Request

## Related issue
Fixes #819 #820 #821 

## What does this PR do?
- Adds `usedDatabaseSize` to stats
- Adds documents database metrics to stats
- Adds embbedings database metrics to stats

## PR checklist
Please check if your PR fulfills the following requirements:
- [x] Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
- [x] Have you read the contributing guidelines?
- [x] Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!


Co-authored-by: Prateek Sharma <[email protected]>
Co-authored-by: Prateek Sharma <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Strift