Re: [sqlalchemy] Purpose of assertion in table reflection test (1.1.9)

2017-06-21 Thread jan.karstens via sqlalchemy
Quick feedback - with your input I managed to get the test cases working. There was one path in the code where the denormalization was not applied, there was an issue with the default schema name, and there was an issue with code pages. As always an interesting bag of root causes. Again, thank

Re: [sqlalchemy] Purpose of assertion in table reflection test (1.1.9)

2017-05-30 Thread jan.karstens via sqlalchemy
Thank you so much for the detailed answer. Very much appreciated, this should get me on the right track to fix this for the eXAsoL dialect. The ExaSolution database (this is the original product name, Exasol being the company name - but no user cares and calls the DB Exasol) was originally a

Re: [sqlalchemy] Purpose of assertion in table reflection test (1.1.9)

2017-05-19 Thread mike bayer
On 05/18/2017 06:56 PM, jan.karstens via sqlalchemy wrote: Upgrading a specific dialect (EXASOL) to 1.1.9 made me stumble across this test (part of test_reflection.py): https://github.com/zzzeek/sqlalchemy/blob/rel_1_1_9/lib/sqlalchemy/testing/suite/test_reflection.py#L737 def

Re: [sqlalchemy] Purpose of assertion in table reflection test (1.1.9)

2017-05-19 Thread Simon King
On Thu, May 18, 2017 at 11:56 PM, jan.karstens via sqlalchemy < sqlalchemy@googlegroups.com> wrote: > Upgrading a specific dialect (EXASOL) to 1.1.9 made me stumble across this > test (part of test_reflection.py): > > https://github.com/zzzeek/sqlalchemy/blob/rel_1_1_9/lib/ >

[sqlalchemy] Purpose of assertion in table reflection test (1.1.9)

2017-05-18 Thread jan.karstens via sqlalchemy
Upgrading a specific dialect (EXASOL) to 1.1.9 made me stumble across this test (part of test_reflection.py): https://github.com/zzzeek/sqlalchemy/blob/rel_1_1_9/lib/sqlalchemy/testing/suite/test_reflection.py#L737 def test_get_table_names(self): tablenames = [ t for t in