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 MS SQL Server Dialect #53

Closed
heenenee opened this issue Sep 19, 2016 · 10 comments
Closed

Add MS SQL Server Dialect #53

heenenee opened this issue Sep 19, 2016 · 10 comments
Assignees

Comments

@heenenee
Copy link

I'm trying to perform a limit query against a Microsoft SQL Server database, but I'm getting a "No dialect registered for sqlserver" error. Exposed could use a SQLServerDialect class.

@Tapac
Copy link
Contributor

Tapac commented Sep 21, 2016

Sorry, but we don't have enough experience with MSSQL and MSSQL Compact for unit test, so if have - please PR SQLServerDialect. You can find examples in H2.kt, Mysql.kt

@Tapac Tapac mentioned this issue Aug 14, 2017
YakovSirotkin pushed a commit to YakovSirotkin/Exposed that referenced this issue Sep 14, 2017
YakovSirotkin pushed a commit to YakovSirotkin/Exposed that referenced this issue Sep 14, 2017
YakovSirotkin pushed a commit to YakovSirotkin/Exposed that referenced this issue Sep 14, 2017
YakovSirotkin pushed a commit to YakovSirotkin/Exposed that referenced this issue Sep 14, 2017
YakovSirotkin pushed a commit to YakovSirotkin/Exposed that referenced this issue Sep 14, 2017
YakovSirotkin pushed a commit to YakovSirotkin/Exposed that referenced this issue Sep 14, 2017
YakovSirotkin pushed a commit to YakovSirotkin/Exposed that referenced this issue Sep 14, 2017
YakovSirotkin pushed a commit to YakovSirotkin/Exposed that referenced this issue Sep 14, 2017
YakovSirotkin pushed a commit to YakovSirotkin/Exposed that referenced this issue Sep 14, 2017
YakovSirotkin pushed a commit to YakovSirotkin/Exposed that referenced this issue Sep 14, 2017
YakovSirotkin pushed a commit to YakovSirotkin/Exposed that referenced this issue Sep 14, 2017
YakovSirotkin pushed a commit to YakovSirotkin/Exposed that referenced this issue Sep 14, 2017
@YakovSirotkin
Copy link
Contributor

YakovSirotkin commented Sep 14, 2017

I added support for MSSQL in https://github.com/YakovSirotkin/Exposed/tree/sqlserver-dialect, but I want to wait with pull request until clarification with #155

Testing with Docker is not trivial and maybe it will be possible to re-use approach for Oracle tests to SQL Server.

YakovSirotkin pushed a commit to YakovSirotkin/Exposed that referenced this issue Sep 14, 2017
YakovSirotkin pushed a commit to YakovSirotkin/Exposed that referenced this issue Sep 15, 2017
YakovSirotkin pushed a commit to YakovSirotkin/Exposed that referenced this issue Sep 15, 2017
YakovSirotkin pushed a commit to YakovSirotkin/Exposed that referenced this issue Sep 15, 2017
YakovSirotkin pushed a commit to YakovSirotkin/Exposed that referenced this issue Sep 15, 2017
YakovSirotkin pushed a commit to YakovSirotkin/Exposed that referenced this issue Sep 15, 2017
@YakovSirotkin
Copy link
Contributor

I created pull request #164 with SQL Server support, but I am not sure about Docker tests. Maybe it makes sense to create one tests with 2 Docker instance with Oracle and SQL Server.

@Tapac
Copy link
Contributor

Tapac commented Sep 15, 2017

Ok, I will check PR next week and will try to setup single TC configuration to run both DBMS.

@mkporwit
Copy link

Would be great to get MS SQL support in Exposed. We're about to start integrating our services with our Azure backend. Happy to help with testing/validation if that's of benefit.

YakovSirotkin pushed a commit to YakovSirotkin/Exposed that referenced this issue Sep 20, 2017
YakovSirotkin pushed a commit to YakovSirotkin/Exposed that referenced this issue Sep 20, 2017
@YakovSirotkin
Copy link
Contributor

@Tapac Do you have a plan how to add SQL Server support? Maybe we can create branch in main repo from #164 (comment)? At least it will be easier to keep branch up-to-date.

Tapac pushed a commit that referenced this issue Oct 6, 2017
* #53 namedEmptyTableWithoutQuotesSQL test works only with H2
* #53 Add alreadyOrdered parameter to limit function for SQLServer support.
* #53 SQLServer support
* #53 Provide additional parameter to limit() that shows if we have ORDER BY clause already.
* #53 Workaround for SQLServer that do not return generated keys after batch insert.
* #53 Workaround for SQLServer that requires special blob handling.
* #53 Disable time-related tests for SQLServer where it has no chance.
* #53 Register SQLServer dialect.
* #53 Workaround for SQLServer that looses precision when it stores milliseconds.
* #53 Add SQLServer driver dependency.
* #53 Add SQLServer support.
* #53 Update for limitations section.
* #53 Use currentDialect.
* #53 Driver dependencies are fixed already.
* #53 Add property for dialect for Docker tests.
* #53 Update instructions how to start Docker.
* #53 Use system properties for SQLSERVER configuration.
* #53 Support Docker tests for Oracle and SQLServer.
* #53 Switch to SQL Server JDBC driver for Java 7.
* #53 Remove Oracle-specific Docker properties.
@Tapac
Copy link
Contributor

Tapac commented Nov 13, 2017

@YakovSirotkin, finally I'm ready to merge your branch with SQLDialect into master and create a release.
First of all, thank you for that PR, and let me know you have an account at kotlinlang.slack.com, because I want to mention you in release announce in the Exposed channel.

Also, I found out that DATETIME2 type doesn't have that problem with losing millis.

@YakovSirotkin
Copy link
Contributor

@Tapac thanks a lot for all your work! I think that I don't have account at kotlinlang.slack.com because I don't have email at jetbrains.com:)

@Tapac
Copy link
Contributor

Tapac commented Nov 14, 2017

@YakovSirotkin , I'm not sure that it's required to, but sent you invitation to @telamon.ru anyway .

@YakovSirotkin
Copy link
Contributor

@Tapac, thanks a lot, I just registered.

@Tapac Tapac self-assigned this Nov 14, 2017
@Tapac Tapac closed this as completed Nov 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants