Skip to content

Fix #2468 alterCollection, approach 1 (like "createCollection") -- secondary option#2470

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

Fix #2468 alterCollection, approach 1 (like "createCollection") -- secondary option#2470
tatu-at-datastax wants to merge 23 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

github-actions Bot commented May 14, 2026

Copy link
Copy Markdown
Contributor

📉 Unit Test Coverage Delta vs Main Branch

Metric Value
Main Branch 52.50%
This PR 52.33%
Delta 🔴 -0.18%
⚠️ Coverage decreased

@github-actions

github-actions Bot commented May 14, 2026

Copy link
Copy Markdown
Contributor

Unit Test Coverage Report

Overall Project 52.33% -0.48% 🍏
Files changed 12.8%

File Coverage
SchemaException.java 100% 🍏
CommandName.java 97.31% 🍏
InsertCollectionOperation.java 94.43% -1.28%
CreateCollectionOperation.java 69.18% -1.11% 🍏
CollectionLexicalDef.java 22.4% -22.93%
CollectionDriverExceptionHandler.java 11.76% 🍏
AlterCollectionCommand.java 0%
CreateCollectionCommandResolver.java 0% -1.33%
AlterCollectionCommandResolver.java 0%
CollectionResource.java 0% 🍏
AlterCollectionLexicalOperation.java 0%

@github-actions

github-actions Bot commented May 14, 2026

Copy link
Copy Markdown
Contributor

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

Metric Value
Main Branch 72.50%
This PR 72.24%
Delta 🔴 -0.26%
⚠️ Coverage decreased

@github-actions

github-actions Bot commented May 14, 2026

Copy link
Copy Markdown
Contributor

Integration Test Coverage Report (dse69-it)

Overall Project 72.24% -0.45% 🍏
Files changed 18.69%

File Coverage
CommandName.java 100% 🍏
SchemaException.java 100% 🍏
CreateCollectionCommandResolver.java 96.94% 🍏
CollectionResource.java 96.8% 🍏
InsertCollectionOperation.java 91.43% -1.28%
CollectionDriverExceptionHandler.java 83.53% 🍏
AlterCollectionCommand.java 80% -20% 🍏
CreateCollectionOperation.java 72% -5.74%
CollectionLexicalDef.java 56% -9.33%
AlterCollectionCommandResolver.java 19.73% -80.27%
AlterCollectionLexicalOperation.java 0%

@github-actions

github-actions Bot commented May 14, 2026

Copy link
Copy Markdown
Contributor

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

Metric Value
Main Branch 73.88%
This PR 73.96%
Delta 🟢 +0.07%
✅ Coverage improved!

@github-actions

github-actions Bot commented May 14, 2026

Copy link
Copy Markdown
Contributor

Integration Test Coverage Report (hcd-it)

Overall Project 73.96% -0.05% 🍏
Files changed 91.7% 🍏

File Coverage
CommandName.java 100% 🍏
SchemaException.java 100% 🍏
CreateCollectionCommandResolver.java 96.94% 🍏
CollectionResource.java 96.8% 🍏
InsertCollectionOperation.java 94.65% 🍏
CollectionLexicalDef.java 93.87% 🍏
AlterCollectionCommandResolver.java 93.2% -6.8% 🍏
CollectionDriverExceptionHandler.java 91.76% 🍏
AlterCollectionLexicalOperation.java 85.12% -14.88% 🍏
AlterCollectionCommand.java 80% -20% 🍏
CreateCollectionOperation.java 79.28% 🍏

@tatu-at-datastax tatu-at-datastax changed the title Fix #2468 alterCollection, approach 1 Fix #2468 alterCollection, approach 1 (like "createCollection") May 19, 2026
@tatu-at-datastax tatu-at-datastax changed the title Fix #2468 alterCollection, approach 1 (like "createCollection") Fix #2468 alterCollection, approach 1 (like "createCollection") -- secondary option May 22, 2026
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