-
Notifications
You must be signed in to change notification settings - Fork 0
AJDA-214: Update ODBC driver #44
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
base: master
Are you sure you want to change the base?
Conversation
…es fetch() method - mainly to set the odbc_longreadlen property
$parameters['DefaultStringColumnLength'] = '16777216'; | ||
$parameters['DefaultVarcharColumnLength'] = '16777216'; | ||
$parameters['BinaryColumnLength'] = '16777216'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Timhle si nejsem jistej, jestli to ma opravdu vlyv, ale funguje to takhle, uz to tak necham.
Kdyby se objevil problem, tak to muzem postelovat
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
jinak dobrý, ale ty datadir testy by se měly pouštět
https://keboola.atlassian.net/browse/AJDA-214
Story
The extractor was failing on segmentation fault --> updated driver (use Debian one, not RPM)
The new driver was throwing exception on getting primary keys --> needed to update db-extractor-adapter keboola/db-extractor-adapter#35
The I was experimenting with settings for odbc_longreadlen, suggested by AI, which helped when I put large enough number there :)
The last problem was corrupted utf-8 character, so I put utf8 sanitization into the extractor itself.
Changes