gordthompson / sqlalchemy-sybase

SAP ASE (Sybase) support for SQLAlchemy implemented as an external dialect
MIT License
12 stars 4 forks source link

BUG: supports_sane_rowcount = False #4

Closed snowman2 closed 4 years ago

snowman2 commented 4 years ago

Closes #3

This setting change seems to fix the issue.

gordthompson commented 4 years ago

Thanks, Alan. I flipped that bit to True back in May because a lot of the "sane_rowcount" stuff seemed to be related to issues with much older versions of pyodbc, and my initial testing with current pyodbc and SAP ASE ODBC seemed fine with it. Sorry for the inconvenience.

snowman2 commented 4 years ago

That makes sense - no worries. Most things seemed to work, just this random hiccup. The pyodbc version I used for testing was the current latest 4.0.30. I wonder if it has to do with the database Adaptive Server Enterprise/16.0 SP02 I am using?

gordthompson commented 4 years ago

I wonder if it has to do with the database Adaptive Server Enterprise/16.0 SP02 I am using?

Seems unlikely to me. These rowcount issues have been around for a long time and they strike me as the sort of thing that is so baked into the backend (or the ODBC driver, or both) that they probably won't get fixed on their own, but if they do happen to get fixed as a side-effect of some other change most people won't notice because they're already using some sort of workaround anyway.

snowman2 commented 4 years ago

Interesting :thinking:. Thanks for the quick release :+1: