add more support for informix ids server

Issue #1906 resolved
Former user created an issue

Hi,

I cleaned up (more or less) the informixdb adapter to make it working in sa 0.6. It's not complete yet, but after seeing 992dd2c055dcff75953695d20813f43d858997d9 I thought it would be wise to add it to sa core now, before others do the same work... I am testing on v10 for now, will switch to v11 soon (though v7 appears to be working too ;)).

Cheers, Florian Apolloner

Comments (5)

  1. Former user Account Deleted

    Btw: The patch alsoreplaces count(1) in the tests with count(*) as the first one won't work on informix (at least from what I can tell…) It also replaces "-" with "_" in column names to get the tests pass. Following jek I also replaced:

            Column('test_passivedefault2', sa.Integer, server_default='5'),
    
        Column('test_passivedefault2', sa.Integer, server_default=sa.text('5')),
    

    }}}

  2. Former user Account Deleted

    New patch uses a regex to extract the correct version ('11.50xC3' becomes (11, 50, 'xC3') now instead of (11, '50xC3')). It also reverts the server_default back to '5' instead of sa.text('5'), InformixDDLcompiler now takes care of that.

    Cheers, Florian Apolloner (apollo13)

  3. Mike Bayer repo owner

    Looks fantastic. I can't say that it works since I can't test, but I at least got the "compile" test going without the backend enabled.

    If you wanted to continue on this, we'd need to fill up test/dialect/test_informix.py with tests and also add documentation to both informix/base.py and informix/informixdb.py regarding special behaviors, notes, caveats.

    761b4f29e295cfaa774a9a2fec713f9055548387.

  4. Log in to comment