lbitonti / liquibase-hana

Liquibase Hana extension
Apache License 2.0
11 stars 11 forks source link

Contributing our team's modifications to the HANA extension #3

Closed sabineheider closed 9 years ago

sabineheider commented 9 years ago

Hi Luigi,

we have been using the HANA extension in our project for about a year now - thanks for providing and maintaining it.

Our use case required a few modifications, which I'd like to contribute back now. I preserved the individual commits (rather than merging them into one big change), so that there are still a reasonable history and meaningful commit messages. Our changes comprise:

We only modified what we actually used in our project. We did not correct all possible ALTER TABLE statements, for example, but only those where we actually got SQL exceptions.

Please have a look at our work and check whether you'd like to merge it into the master branch.

Thanks and best regards, Sabine

lbitonti commented 9 years ago

Hi Sabine, Many thanks for your email and the pull request. Hopefully I'll manage to find some time to review and merge your changes soon. Best Regards,Luigi

 On Thursday, April 9, 2015 9:44 AM, Sabine Heider <notifications@github.com> wrote:

Hi Luigi,we have been using the HANA extension in our project for about a year now - thanks for providing and maintaining it. Our use case required a few modifications, which I'd like to contribute back now. I preserved the individual commits (rather than merging them into one big change), so that there are still a reasonable history and meaningful commit messages. Our changes comprise:

File Changes

Patch Links:

lbitonti commented 9 years ago

Merged. Thank you!