Support ODBC connections

multiprocessio / datastation

App to easily query, script, and visualize data from every database, file, and API.

https://datastation.multiprocess.io

Other

2.9k stars 112 forks source link

Support ODBC connections #273

Closed Posrabi closed 2 years ago

Posrabi commented 2 years ago

Support ODBC databases.
Use a Go plugin around unixODBC to prevent runtime dependency
Add ODBC tests.

eatonphil commented 2 years ago

Sweet nice work! BTW you need to copy the change to the desktop.build script to the same file in the server directory too.

eatonphil commented 2 years ago

Regarding the failing test:

Add this directory to .prettierignore

Posrabi commented 2 years ago

Regarding the failing test:

Add this directory to .prettierignore

I think we could just delete it because the library is already saved to /usr/local/lib

eatonphil commented 2 years ago

Sure, whatever you'd like

eatonphil commented 2 years ago

Does this change what is needed to develop locally btw? Does everyone who wants to run yarn start-desktop need to have unixODBC installed?

Posrabi commented 2 years ago

Yes, libodbc.a will need to be present on ld's search path to build it. Though I did try to remove all odbc related stuff and it still compiles and runs so could you double check on your computer?

eatonphil commented 2 years ago

Man I'm sorry, I should have looked at the license. We can't do static linking because unixodbc is gpl.