Skip to content

quick fixes to follow #2380#2384

Merged
amorton merged 1 commit intomainfrom
ajm/remove-v1-error-tidy
Feb 10, 2026
Merged

quick fixes to follow #2380#2384
amorton merged 1 commit intomainfrom
ajm/remove-v1-error-tidy

Conversation

@amorton
Copy link
Contributor

@amorton amorton commented Feb 10, 2026

the work in #2380 was to replace old with new error, this tidies a couple of things up

What this PR does:

Which issue(s) this PR fixes:
Fixes #

Checklist

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

the work in #2380 was to replace old with new error, this
tidies a couple of things up
@amorton amorton requested a review from a team as a code owner February 10, 2026 00:49
@github-actions
Copy link
Contributor

➡️ Unit Test Coverage Delta vs Main Branch

Metric Value
Main Branch 50.27%
This PR 50.27%
Delta ⚪ 0.00%
ℹ️ Coverage unchanged

@github-actions
Copy link
Contributor

Unit Test Coverage Report

Overall Project 50.27% 🍏
File Coverage
DeleteCollectionOperation.java 98.88% 🍏
NoOptionsCommand.java 97.56% 🍏
EmbeddingProviderResponseValidation.java 90.11% 🍏
PathMatchLocator.java 87.71% 🍏
DocumentId.java 79.77% 🍏
CommandObjectMapperHandler.java 26.17% 🍏

Copy link
Contributor

@tatu-at-datastax tatu-at-datastax left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions
Copy link
Contributor

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

Metric Value
Main Branch 70.73%
This PR 70.73%
Delta ⚪ 0.00%
ℹ️ Coverage unchanged

@github-actions
Copy link
Contributor

Integration Test Coverage Report (dse69-it)

Overall Project 70.73% 🍏
File Coverage
NoOptionsCommand.java 100% 🍏
CommandObjectMapperHandler.java 98.66% 🍏
DeleteCollectionOperation.java 95.74% 🍏
PathMatchLocator.java 72.73% 🍏
DocumentId.java 62.5% 🍏
EmbeddingProviderResponseValidation.java 25.27% 🍏

@github-actions
Copy link
Contributor

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

Metric Value
Main Branch 72.48%
This PR 72.48%
Delta ⚪ 0.00%
ℹ️ Coverage unchanged

@github-actions
Copy link
Contributor

Integration Test Coverage Report (hcd-it)

Overall Project 72.48% 🍏
File Coverage
NoOptionsCommand.java 100% 🍏
CommandObjectMapperHandler.java 98.66% 🍏
DeleteCollectionOperation.java 95.74% 🍏
PathMatchLocator.java 72.73% 🍏
DocumentId.java 62.5% 🍏
EmbeddingProviderResponseValidation.java 25.27% 🍏

@amorton amorton merged commit 86c47a9 into main Feb 10, 2026
3 checks passed
@amorton amorton deleted the ajm/remove-v1-error-tidy branch February 10, 2026 02:07
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.

2 participants