Skip to content
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

Adjust TDS Version used by pymsql #78

Open
SenneVanstraelen opened this issue Aug 6, 2024 · 3 comments
Open

Adjust TDS Version used by pymsql #78

SenneVanstraelen opened this issue Aug 6, 2024 · 3 comments

Comments

@SenneVanstraelen
Copy link

Hello

I need to be able to connect to SQL Server 2005. For this I need to adjust the TDS Version, like in the variant by Rob Winters.

Is there a way to do this? I cannot seem to find it in the documentation.

Thanks!

@BuzzCutNorman
Copy link
Owner

Sorry to say but buzzcutnorman tap-mssql does not currently have a way to tell pymssql the TDS version you want it to use.

I took a look at pipelinewise-tap-mssql by Rob Winters. He uses pymssql exclusively and can pass that directly during the connection creation. The singer-sdk I used for creating buzzcutnorman tap-msql uses pymssql in conjunction with SQLAlchemy. After a quick search I didn't find a quick way to add this. I will look into it more. This tap also supports pyodbc which might be able to connect to SQL Server 2005.

Also, if pipelinewise-tap-mssql by Rob Winters has the functionality you need by all means use it. It is the default tap-mssql on hub.meltano.com because it is production ready and battle tested.

@SenneVanstraelen
Copy link
Author

Thanks for looking into it. With the pipelinewise-tap-mssql by Rob Winters I'm experiencing other issues related to the table selection and the version of the database and TDS.

The ODBC route is worth looking into, however, I need to find a driver that works on Linux and can connect to SQL Server 2005 (TDS 7.0). That information doesn't seem to be readily available on the internet, any ideas on which driver to use, I know for a fact that 18 does not work.

@BuzzCutNorman
Copy link
Owner

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants