<p dir="ltr">Hi Konrad,<br>
No problem!<br>
I should have brought it up before but unfortunately I realized the need for an 'alter' only after the test I wrote didn't pass :)</p>
<p dir="ltr">In my opinion, as Thomas mentioned we should/could 'cheat' this time and run the database SQL directly on the database Instead of doing the same via the framework we are planning to put in place.</p>
<p dir="ltr">Thanks,<br>
Shoubhik </p>
<div class="gmail_extra"><br><div class="gmail_quote">On Sep 25, 2016 7:11 PM, "Baiju Muthukadan" <<a href="mailto:bmuthuka@redhat.com">bmuthuka@redhat.com</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Max,<br>
<div class="quoted-text"><br>
On Sun, Sep 25, 2016 at 4:44 PM, Max Andersen <<a href="mailto:manderse@redhat.com">manderse@redhat.com</a>> wrote:<br>
> Why fork ? Is it not enough to just not use the Cassandra parts ?<br>
><br>
> Also, wasn't a similar library found and it was rejected since it only<br>
> allowed .sql migrations where we wanted go based migrations too ?<br>
<br>
</div>As per earlier discussion, there were three reasons for not choosing<br>
the other project:<br>
<a href="https://github.com/almighty/almighty-core/issues/184#issuecomment-246901574" rel="noreferrer" target="_blank">https://github.com/almighty/<wbr>almighty-core/issues/184#<wbr>issuecomment-246901574</a><br>
<br>
> 1. We only have to support one database (PostgreSQL) and not the many others supported by the project.<br>
<br>
This is one the reason that I forked the project.  Because the<br>
original author may not be interested to remove features.<br>
<br>
> 2. We want to have pre- and post-migration scripts (probably in Go) that need to run smart migration steps, that would otherwise be too cumbersome to write in pure SQL files.<br>
<br>
I have implemented this feature. See the documentation:<br>
<a href="https://godoc.org/github.com/baijum/pgmigration" rel="noreferrer" target="_blank">https://godoc.org/github.com/<wbr>baijum/pgmigration</a><br>
<br>
If the first reason for forking can be ignored, I can send this<br>
feature to the original project and we can use that. Please let me<br>
know.<br>
<br>
> 3. We don't want to support down-migrations.<br>
<br>
The current package doesn't have down-migrations.<br>
<div class="elided-text"><br>
Regards,<br>
Baiju M<br>
<br>
______________________________<wbr>_________________<br>
almighty-public mailing list<br>
<a href="mailto:almighty-public@redhat.com">almighty-public@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/almighty-public" rel="noreferrer" target="_blank">https://www.redhat.com/<wbr>mailman/listinfo/almighty-<wbr>public</a><br>
</div></blockquote></div><br></div>