Skip to content

Implement #2456: include model name tag in vectorize metrics#2457

Merged
tatu-at-datastax merged 3 commits into
mainfrom
tatu/2456-model-name-in-vectorize-metrics
Apr 23, 2026
Merged

Implement #2456: include model name tag in vectorize metrics#2457
tatu-at-datastax merged 3 commits into
mainfrom
tatu/2456-model-name-in-vectorize-metrics

Conversation

@tatu-at-datastax
Copy link
Copy Markdown
Contributor

What this PR does:

As title says, adds "embedding.model" tag in vectorize metrics

Which issue(s) this PR fixes:
Fixes #2456

Checklist

  • Changes manually tested
  • Automated Tests added/updated
  • Documentation added/updated
  • CLA Signed: DataStax CLA

@tatu-at-datastax tatu-at-datastax requested a review from a team as a code owner April 21, 2026 20:30
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 21, 2026

📈 Unit Test Coverage Delta vs Main Branch

Metric Value
Main Branch 49.79%
This PR 49.80%
Delta 🟢 +0.01%
✅ Coverage improved!

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 21, 2026

Unit Test Coverage Report

Overall Project 49.8% 🍏
Files changed 100% 🍏

File Coverage
JsonApiMetricsConfig.java 100% 🍏
MeteredEmbeddingProviderWrapper.java 97.42% 🍏

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 21, 2026

📈 Integration Test Coverage Delta vs Main Branch (dse69-it)

Metric Value
Main Branch 72.66%
This PR 72.66%
Delta 🟢 +0.00%
✅ Coverage improved!

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 21, 2026

Integration Test Coverage Report (dse69-it)

Overall Project 72.66% 🍏
Files changed 97.22% 🍏

File Coverage
JsonApiMetricsConfig.java 100% 🍏
MeteredEmbeddingProviderWrapper.java 97.05% -0.37% 🍏

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 21, 2026

📈 Integration Test Coverage Delta vs Main Branch (hcd-it)

Metric Value
Main Branch 74.04%
This PR 74.05%
Delta 🟢 +0.00%
✅ Coverage improved!

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 21, 2026

Integration Test Coverage Report (hcd-it)

Overall Project 74.05% 🍏
Files changed 97.22% 🍏

File Coverage
JsonApiMetricsConfig.java 100% 🍏
MeteredEmbeddingProviderWrapper.java 97.05% -0.37% 🍏

@tatu-at-datastax tatu-at-datastax self-assigned this Apr 22, 2026
@tatu-at-datastax tatu-at-datastax merged commit c346269 into main Apr 23, 2026
3 checks passed
@tatu-at-datastax tatu-at-datastax deleted the tatu/2456-model-name-in-vectorize-metrics branch April 23, 2026 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Include Model Name (not just Embedding Provider) tag for Vectorizer metrics

2 participants