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

Add support to MSSQL/TSQL (Azure) #934

Closed
ipepe opened this issue Oct 26, 2015 · 5 comments
Closed

Add support to MSSQL/TSQL (Azure) #934

ipepe opened this issue Oct 26, 2015 · 5 comments

Comments

@ipepe
Copy link

ipepe commented Oct 26, 2015

Hello.
Im running rails app on Jruby with MSSQL database but geocoder deosnt support that. Queries have ATAN2 instead of ATN2 of MSSQL. If You need a database I can setup one in azure for the matter of working on that. I You need my help, just tell me.

@alexreisner
Copy link
Owner

Sorry, but I feel similarly about MSSQL as about Oracle (see #535). These databases are not especially popular in the Ruby community and the effort required to add compatibility is quite significant. Only way I could consider adding is if someone submitted a clean pull request (minimal code changes) that included the necessary updates to Travis CI config. Even then, I'd really have to weight the depth and complexity of the code changes against the benefits.

@ipepe
Copy link
Author

ipepe commented Oct 26, 2015

What about doing it like sqlite?

@alexreisner
Copy link
Owner

I don't like how SQLite is handled currently. Ideally there would be separate drivers for each DB system. The way SQLite is currently implemented it's a special case, handled mostly by if statements. I don't like the idea of introducing more branches (because it's not good code) or restructuring the whole thing (better code, but deep changes) for functionality that's not core.

@ipepe
Copy link
Author

ipepe commented Oct 27, 2015

based on this:
http://stackoverflow.com/questions/13026675/calculating-distance-between-two-points-latitude-longitude
I declare that I want to add support to MSSQL. I will create pull request. But before I dive into code, is there a chance for those separate adapters, or maybe just a stub of that?

@alexreisner
Copy link
Owner

OK, you're welcome to submit a PR but just to be clear: there is a good chance I won't merge because I can't currently think of an implementation that I feel comfortable with. Even if not merged, your work will still be useful because you'll be able to install the gem from your own repo. Just wanted to be clear about this.

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