Skip to content
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

Informix support #104

Closed
flyway opened this issue Jun 25, 2013 · 23 comments
Closed

Informix support #104

flyway opened this issue Jun 25, 2013 · 23 comments

Comments

@flyway
Copy link
Collaborator

@flyway flyway commented Jun 25, 2013

Original author: Tomislav...@gmail.com (August 19, 2011 15:19:48)

Is there any possibility to support Informix database? Thx

Original issue: http://code.google.com/p/flyway/issues/detail?id=155

@trajakovic
Copy link

@trajakovic trajakovic commented Jun 25, 2013

+1

@kratochj
Copy link

@kratochj kratochj commented Nov 27, 2013

Any progress?

@axelfontaine
Copy link
Contributor

@axelfontaine axelfontaine commented Nov 28, 2013

This is not a priority at this point. Pull request welcome though.

@rigareau
Copy link

@rigareau rigareau commented Jun 10, 2014

+1

2 similar comments
@lukedive
Copy link

@lukedive lukedive commented Jun 18, 2014

+1

@andersmoe
Copy link

@andersmoe andersmoe commented Aug 13, 2014

+1

@noerstad
Copy link

@noerstad noerstad commented Sep 5, 2016

+1

Any news on if this will be prioritized in the near future or if we have to start looking at "competing" frameworks? ;)

@ogjerstad
Copy link

@ogjerstad ogjerstad commented Sep 13, 2016

+1

2 similar comments
@belingueres
Copy link

@belingueres belingueres commented Oct 5, 2016

+1

@ceinmart
Copy link

@ceinmart ceinmart commented Nov 10, 2016

+1

@struesda
Copy link

@struesda struesda commented Dec 13, 2016

is there any documentation on the steps to add support for a DB? We are considering FlywayDB but will need to add Informix support to fully leverage it.

@carljmosca
Copy link

@carljmosca carljmosca commented Dec 21, 2016

Given that dbaccess is required for certain changes (such as creating stored procedures), would this not be of limited use?

@belingueres
Copy link

@belingueres belingueres commented Dec 22, 2016

You can create stored procedures using the JDBC driver (I do it with dbVisualizer client tool). The only thing I coudn't do with a client tool is to get the query plan since Informix stores it in a text file on the server filesystem.

@carljmosca
Copy link

@carljmosca carljmosca commented Dec 22, 2016

Which version of the driver are you using? I also use DbVis and it is currently failing for me. I assume this is outdated since it's working for you: http://www-01.ibm.com/support/docview.wss?uid=swg21689642

@belingueres
Copy link

@belingueres belingueres commented Dec 22, 2016

@carljmosca
Copy link

@carljmosca carljmosca commented Dec 23, 2016

Ah, I think I now see my confusion. Are you able to use the "execute the complete buffer as one SQL statement" for complex DDL including create table and triggers along with some DML?

I have also used DbVis for years for individual DDL and some more complex but have not currently found a way to execute complex ones (DDL and DML).

@belingueres
Copy link

@belingueres belingueres commented Dec 23, 2016

@carljmosca
Copy link

@carljmosca carljmosca commented Dec 23, 2016

I am not saying it's not possible. I was asking for an example. I don't think it's a showstopper anyhow.

@carljmosca
Copy link

@carljmosca carljmosca commented Dec 24, 2016

I have submitted a pull request with some initial, very limited support for Informix from here: https://github.com/carljmosca/flyway

@carljmosca
Copy link

@carljmosca carljmosca commented Dec 26, 2016

I have now added and tested support for the DELIMITER in the Informix support. The example I asked for above can now be achieved with the DELIMITER.

For example:

DELIMITER $$
create trigger "informix".my_trigger delete on
"informix".some_table referencing old as old
for each row
(
execute procedure "informix".special_update(old.some_id
,NULL ,NULL ));
$$

A second (or third) set of eyes would be great.

@davalente
Copy link

@davalente davalente commented Jun 22, 2017

+1

@carljmosca
Copy link

@carljmosca carljmosca commented Jun 22, 2017

thank you - I believe I addressed this in my fork

the pull request was rejected due to tests which apparently I don't understand because last I checked they worked for me

@axelfontaine axelfontaine added this to the Flyway 5.1.0 milestone Nov 26, 2017
axelfontaine added a commit to flyway/flywaydb.org that referenced this issue May 7, 2018
@axelfontaine
Copy link
Contributor

@axelfontaine axelfontaine commented May 7, 2018

This has now been implemented.

dohrayme pushed a commit to dohrayme/flyway that referenced this issue Feb 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet