add more support for informix ids server
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)
-
Account Deleted -
repo owner -
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)
-
repo owner - changed status to resolved
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.
-
repo owner - removed milestone
Removing milestone: 0.6.5 (automated comment)
- Log in to comment
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:
}}}