Skip to content

Fix #2468 alterCollection, approach 1#2470

Draft
tatu-at-datastax wants to merge 13 commits into
mainfrom
tatu/2468-alter-collection-approach-1
Draft

Fix #2468 alterCollection, approach 1#2470
tatu-at-datastax wants to merge 13 commits into
mainfrom
tatu/2468-alter-collection-approach-1

Conversation

@tatu-at-datastax
Copy link
Copy Markdown
Contributor

What this PR does:

Implements "alterCollection" command to allow enabling of "lexical" indexing, with "createCollection" style

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

Checklist

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

@tatu-at-datastax tatu-at-datastax self-assigned this May 14, 2026
@github-actions
Copy link
Copy Markdown
Contributor

📉 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

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

📉 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

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

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

Metric Value
Main Branch 74.21%
This PR 74.33%
Delta 🟢 +0.12%
✅ Coverage improved!

@github-actions
Copy link
Copy Markdown
Contributor

Integration Test Coverage Report (hcd-it)

Overall Project 74.33% -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% 🍏

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