Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Gracefully handle search index exceptions on non MongoDB Atlas #2673

Merged

Conversation

Jafarili
Copy link
Contributor

Allow users to explicitly exclude features (e.g. search indexes) in CLI commands.

Q A
Type bug
BC Break no
Fixed issues #2671 (comment)

Summary

The CLI commands for managing schema entities (e.g. creating collections and indexes) attempt to process all schema definitions by default. This is problematic for features such as Atlas Search indexes, which have more restrictions.

For example, CreateCommand defines an ordered list (i.e. $createOrder) for schema entities to process when no command line arguments are specified. If any Atlas Search indexes exist in class metadata, those will be processed by default; however, that may not be supported in a development environment.

If users want to exclude certain DDL operations, they cannot rely on the default order and must manually enumerate all schema entities to process. Consider introducing skip- options to allow users to opt out of specific features.

Context: #2671 (comment)

@alcaeus alcaeus force-pushed the fix-search-index-not-on-mongodb-atlas branch from a72450c to afed019 Compare September 6, 2024 11:41
@alcaeus alcaeus changed the base branch from 2.9.x to 2.8.x September 6, 2024 11:41
@alcaeus alcaeus added this to the 2.8.2 milestone Sep 6, 2024
@alcaeus alcaeus self-assigned this Sep 6, 2024
@alcaeus alcaeus added the Bug label Sep 6, 2024
@alcaeus alcaeus linked an issue Sep 6, 2024 that may be closed by this pull request
@alcaeus alcaeus merged commit 511a476 into doctrine:2.8.x Sep 6, 2024
15 of 17 checks passed
@alcaeus
Copy link
Member

alcaeus commented Sep 6, 2024

Thanks @Jafarili!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Inability to list search indexes is not properly detected
2 participants