Skip to content

Add option to disable automatic index creation#18

Merged
SimonCropp merged 1 commit intomainfrom
Add-option-to-disable-automatic-index-creation
Feb 4, 2026
Merged

Add option to disable automatic index creation#18
SimonCropp merged 1 commit intomainfrom
Add-option-to-disable-automatic-index-creation

Conversation

@SimonCropp
Copy link
Owner

Introduce a createIndexes option to UseDefaultOrderBy (default true) to allow disabling automatic index creation. Propagate the flag through OrderRequiredExtension and UseDefaultOrderByConventionPlugin; add IndexCreationDisabledAnnotation and conditionally register the OrderByIndexConvention. OrderByBuilder now checks the model annotation and throws if WithIndexName() is used when index creation is disabled. Update extension info (log fragment, service provider hash, debug info) to include the new option. Add tests to verify no indexes are created and that WithIndexName() throws when disabled, and update README/snippet references accordingly.

Introduce a createIndexes option to UseDefaultOrderBy (default true) to allow disabling automatic index creation. Propagate the flag through OrderRequiredExtension and UseDefaultOrderByConventionPlugin; add IndexCreationDisabledAnnotation and conditionally register the OrderByIndexConvention. OrderByBuilder now checks the model annotation and throws if WithIndexName() is used when index creation is disabled. Update extension info (log fragment, service provider hash, debug info) to include the new option. Add tests to verify no indexes are created and that WithIndexName() throws when disabled, and update README/snippet references accordingly.
@SimonCropp SimonCropp added this to the 0.1.7 milestone Feb 4, 2026
@SimonCropp SimonCropp merged commit b8d2949 into main Feb 4, 2026
4 of 6 checks passed
@SimonCropp SimonCropp deleted the Add-option-to-disable-automatic-index-creation branch February 4, 2026 05:18
@SimonCropp SimonCropp modified the milestones: 0.1.7, 0.2.0 Feb 4, 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.

1 participant