-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[DAT-15924] Handle AbstractSQLChange runWith checksum v8 variants #4937
Conversation
liquibase-standard/src/main/java/liquibase/util/ValidatingVisitorUtil.java
Fixed
Show fixed
Hide fixed
liquibase-standard/src/main/java/liquibase/util/ValidatingVisitorUtil.java
Fixed
Show fixed
Hide fixed
830473a
to
91e2f9a
Compare
liquibase-standard/src/main/java/liquibase/change/AbstractSQLChange.java
Outdated
Show resolved
Hide resolved
liquibase-standard/src/main/java/liquibase/util/ValidatingVisitorUtil.java
Outdated
Show resolved
Hide resolved
liquibase-standard/src/main/java/liquibase/util/ValidatingVisitorUtil.java
Outdated
Show resolved
Hide resolved
4.19.0 or 4.19.1? Does it matter? |
It's 4.19.1, I confirmed during tests. |
liquibase-standard/src/main/java/liquibase/util/ValidatingVisitorUtil.java
Fixed
Show fixed
Hide fixed
liquibase-standard/src/main/java/liquibase/util/ValidatingVisitorUtil.java
Fixed
Show fixed
Hide fixed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it possible to write any tests around this code?
liquibase-standard/src/main/java/liquibase/util/ValidatingVisitorUtil.java
Fixed
Show fixed
Hide fixed
liquibase-standard/src/main/java/liquibase/util/ValidatingVisitorUtil.java
Fixed
Show fixed
Hide fixed
liquibase-standard/src/main/java/liquibase/util/ValidatingVisitorUtil.java
Fixed
Show fixed
Hide fixed
liquibase-standard/src/main/java/liquibase/util/ValidatingVisitorUtil.java
Fixed
Show fixed
Hide fixed
02ca2f1
to
5936cb7
Compare
Impact
Description
AbstractSqlChange checksum had the checksum calculated value changed for Liquibase versions 4.19.1 to 4.23.1 due to some changes on the way that we call it when using runWith="anything". This PR validates the v8 checksum using the alternative algorithm as a way to allow users to upgrade to checksums v9 without facing any errors or unexpected behaviours