Skip to content

Fix #2468 alterCollection, approach 1#2471

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

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

Conversation

@tatu-at-datastax
Copy link
Copy Markdown
Contributor

What this PR does:

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

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 14, 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 14, 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 14, 2026

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

Metric Value
Main Branch 72.78%
This PR 72.50%
Delta 🔴 -0.28%
⚠️ Coverage decreased

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 14, 2026

Integration Test Coverage Report (dse69-it)

Overall Project 72.5% -0.51% 🍏
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%
CommandObjectMapperHandler.java 91.36% -7.41%
CollectionDriverExceptionHandler.java 83.53% 🍏
AlterCollectionCommand.java 80% -20% 🍏
CreateCollectionOperation.java 70.18% -6.28%
AlterCollectionOperationImpl.java 66.67% -33.33% 🍏
CollectionLexicalConfig.java 42.82% -15.54%
AlterCollectionCommandResolver.java 18.75% -81.25%
AlterCollectionLexicalOperation.java 0%

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 14, 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 14, 2026

Integration Test Coverage Report (hcd-it)

Overall Project 74.28% -0.06% 🍏
Files changed 90.67% 🍏

File Coverage
CommandName.java 100% 🍏
SchemaException.java 100% 🍏
CommandObjectMapperHandler.java 98.77% 🍏
CreateCollectionCommandResolver.java 97.98% 🍏
CollectionResource.java 96.8% 🍏
InsertCollectionOperation.java 94.65% 🍏
CollectionLexicalConfig.java 92.38% 🍏
CollectionDriverExceptionHandler.java 91.76% 🍏
AlterCollectionCommandResolver.java 89.2% -10.8% 🍏
AlterCollectionLexicalOperation.java 85% -15% 🍏
CreateCollectionOperation.java 81.68% 🍏
AlterCollectionCommand.java 80% -20% 🍏
AlterCollectionOperationImpl.java 66.67% -33.33% 🍏

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