drop view before modifying table fields

[pawel-big-lebowski]

Signed-off-by: Pawel Leszczynski leszczynski.pawel@gmail.com

Problem

V52 migration script is failing.

Closes: #2298

Solution

drop view and let it being recreated by flyway after the migration.

Note: All database schema changes require discussion. Please link the issue for context.

Checklist

• You've signed-off your work
• [] Your changes are accompanied by tests (if relevant)
• Your change contains a small diff and is self-contained
• You've updated any relevant documentation (if relevant)
• You've updated the CHANGELOG.md with details about your change under the "Unreleased" section (if relevant, depending on the change, this may not be necessary)
• You've versioned your .sql database schema migration according to Flyway's naming convention (if relevant)
• You've included a header in any source code files (if relevant)

[codecov]

Codecov Report

Merging #2313 (6e07904) into main (68fcb96) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #2313   +/-   ##
=========================================
  Coverage     77.01%   77.01%           
  Complexity     1166     1166           
=========================================
  Files           222      222           
  Lines          5307     5307           
  Branches        424      424           
=========================================
  Hits           4087     4087           
  Misses          747      747           
  Partials        473      473           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[pawel-big-lebowski]

https://flywaydb.org/documentation/concepts/migrations.html#repeatable-migrations

Repeatable migrations have a description and a checksum, but no version. Instead of being run just once, they are (re-)applied every time their checksum changes.

Within a single migration run, repeatable migrations are always applied last, after all pending versioned migrations have been executed. Repeatable migrations are applied in the order of their description.

[fm100]

As the doc says, they are (re-)applied every time their checksum changes. Just dropping the view will not create a view again and the application will break.

[fm100]

#2308

We should drop and recreate a view after altering like above PR.