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

added note in drop unique constraint to specify behavior for dropping index along with constraint in oracleDB #206

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Gh05ts
Copy link

@Gh05ts Gh05ts commented Dec 19, 2023

We recently were caught off guard by this behavior as we explicitly added drop index after the drop constraint in our changeset and the deployment was failing with index does not exist error, so adding to avoid confusion.

https://github.com/liquibase/liquibase/blob/e605cee3c7795688b2d178a67c5fed818ac52d0e/liquibase-standard/src/main/java/liquibase/sqlgenerator/core/DropUniqueConstraintGenerator.java#L18

Had to read the code at line 40 to confirm and we ended up delivering a non-deploying build to QA internally.

If you would prefer to have the note mentioned in some other format or anything along those lines, please let me know or update according to documentation standards.

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.

None yet

1 participant