-
-
Notifications
You must be signed in to change notification settings - Fork 243
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
Support MS SQL Server #40
Comments
Completely agree, it just needs someone to step up and implement it. :) |
Well, I'm interested. Although I probably would need some guidance. |
Of course! I’m happy to offer guidance. @mclark4386 implemented cockroach support recently. I’m sure he’ll be happy to offer some guidance along the way too! |
@markbates So I took a glance at the cockroach commits and if I understand correctly the main task would be to implement sqlserver.go and fizz/translators/sqlserver.go similar to mysql/postgres/cockroach. Is that right? |
That’s the basics of it, yep. I would pick the implementation that’s closest to MS, I’ve never used it, and use that as a starting point. Rename a few things, point them at the MS DB and see what the tests yell at you about. |
@markbates hmm... travis doesn't support ms sql server. would you consider to add appveyor integration? |
Sure. We use it for Buffalo too. Travis also supports docker, if that helps.
…-----------
Mark Bates
On Mar 22, 2018, 4:59 PM -0400, marpio ***@***.***>, wrote:
@markbates hmm... travis doesn't support ms sql server. would you consider to add appveyor integration?
https://www.appveyor.com/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
@markbates true - i've found this: https://github.com/aspnet/EntityFrameworkCore/pull/7842/files |
@marpio happy to help if you have questions! Sounds like you have a good start going. |
@markbates short question - what is the purpose of the {db}_meta.go files in the fiz/translators package? Why doesn't postgres need it? |
Some of the translators, like SQLite (for example), need additional meta information about the table(s), indexes, etc… to create the proper sql for things such as renaming a column. Postgres is a nice db and things just work. :) |
@markbates thanks |
I'm having some difficulties - sqlx doesn't support the sqlserver parameter syntax so:
there is this issue jmoiron/sqlx#374 regarding this problem but it doesn't seem to be active. The second problem is that sqlserver doesn't support the usual LIMIT and OFFSET syntax so i would need to somehow override the Any ideas how this could be solved? |
You probably need to implement your own TranslateSQL function like Postgres does https://github.com/gobuffalo/pop/blob/master/postgresql.go |
I did that but the problem is, sqlx can not map to the fields in an struct, if i pass the params in form '@field_name' instead of ':field_name' so things like:
|
Waiting for jmoiron/sqlx#406 |
@marpio jmoiron/sqlx#406 was merged, and I just merged your PR in fizz. :) |
Awesome @stanislas-m I will need some time to make it work since a lot changed in buffalo/pop since March. |
No problem, take your time! |
@marpio any update on this? |
@Norris1z |
I'm interesting in this. I'm a little green when it comes to Go, and would want to chat about what the scope of work looks like before volunteering to take this on. Gopher slack the best place for that? |
@andyedison Sure, feel free to join the |
It's an common DB in the enterprise and now also available for Linux.
The text was updated successfully, but these errors were encountered: