Skip to content

Initial "alterCollection" implementation#2469

Closed
tatu-at-datastax wants to merge 13 commits into
mainfrom
tatu/2468-alter-collection
Closed

Initial "alterCollection" implementation#2469
tatu-at-datastax wants to merge 13 commits into
mainfrom
tatu/2468-alter-collection

Conversation

@tatu-at-datastax
Copy link
Copy Markdown
Contributor

What this PR does:

Implements "alterCollection" command to allow enabling of "lexical" indexing

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

Checklist

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

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 12, 2026

📉 Unit Test Coverage Delta vs Main Branch

Metric Value
Main Branch 51.62%
This PR 51.44%
Delta 🔴 -0.18%
⚠️ Coverage decreased

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 12, 2026

Unit Test Coverage Report

Overall Project 51.44% -0.48% 🍏
Files changed 15.37%

File Coverage
SchemaException.java 100% 🍏
CommandName.java 97.31% 🍏
InsertCollectionOperation.java 94.43% -1.28%
CreateCollectionOperation.java 68.14% -0.44% 🍏
CollectionLexicalConfig.java 20.53% -25.22%
CollectionDriverExceptionHandler.java 11.76% 🍏
AlterCollectionCommand.java 0%
CreateCollectionCommandResolver.java 0% -2.8%
AlterCollectionCommandResolver.java 0%
CollectionResource.java 0% 🍏
AlterCollectionLexicalOperation.java 0%

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 13, 2026

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

Metric Value
Main Branch 72.78%
This PR 72.53%
Delta 🔴 -0.25%
⚠️ Coverage decreased

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 13, 2026

Integration Test Coverage Report (dse69-it)

Overall Project 72.53% -0.47% 🍏
Files changed 17.57%

File Coverage
CommandName.java 100% 🍏
SchemaException.java 100% 🍏
CreateCollectionCommandResolver.java 97.73% 🍏
CollectionResource.java 96.8% 🍏
InsertCollectionOperation.java 91.43% -1.28%
CollectionDriverExceptionHandler.java 83.53% 🍏
AlterCollectionCommand.java 80% -20% 🍏
CreateCollectionOperation.java 70.18% -6.28%
CollectionLexicalConfig.java 42.82% -15.54%
AlterCollectionCommandResolver.java 20.55% -79.45%
AlterCollectionLexicalOperation.java 0%

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 13, 2026

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

Metric Value
Main Branch 74.21%
This PR 74.28%
Delta 🟢 +0.07%
✅ Coverage improved!

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 13, 2026

Integration Test Coverage Report (hcd-it)

Overall Project 74.28% -0.05% 🍏
Files changed 91.89% 🍏

File Coverage
CommandName.java 100% 🍏
SchemaException.java 100% 🍏
CreateCollectionCommandResolver.java 97.98% 🍏
CollectionResource.java 96.8% 🍏
InsertCollectionOperation.java 94.65% 🍏
AlterCollectionCommandResolver.java 93.15% -6.85% 🍏
CollectionLexicalConfig.java 92.38% 🍏
CollectionDriverExceptionHandler.java 91.76% 🍏
AlterCollectionLexicalOperation.java 85% -15% 🍏
CreateCollectionOperation.java 81.68% 🍏
AlterCollectionCommand.java 80% -20% 🍏

@tatu-at-datastax tatu-at-datastax marked this pull request as ready for review May 14, 2026 15:58
@tatu-at-datastax tatu-at-datastax requested a review from a team as a code owner May 14, 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.

Add command alterCollection, initially to allow enabling "lexical" feature after creation

1 participant