search

ebean-orm / ebean-migration

DB Migration runner (similar to Flyway) which can be used standalone or with Ebean (run migrations on EbeanServer start)
Apache License 2.0
9 stars 5 forks source link

Bad mtype for repeatable scripts #10

Closed thibaultmeyer closed 7 years ago

thibaultmeyer commented 7 years ago

It seems that "mtype" for repeatable scripts is not correct.

Here, an example of SQL files to migrate:

R1__update-sequences.sql
V1__permissions.sql
V2__dns-blaclist.sql
V3__root-account.sql
V4__test-application.sql

db_migration table

image

rbygrave commented 7 years ago

So I think this isn't a bug. I think the issue here is that the above repeatable script starts with "R1" rather than "R". So the 1 is the problem here.

Does that sound correct? You did get past this?

rbygrave commented 7 years ago

Hmm, ok looking to make a change to support "R1", "R2" etc. Note that these don't really match FlywayDB (which has the form of Rcomment) but allowing the R1foo, R2__bar allows the repeatable scripts to have some relative ordering which could be useful.

Looking to push a fix.