-
Notifications
You must be signed in to change notification settings - Fork 26
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
More generic support for SQL Server #12
Comments
I believe that there is something in there that doesn't work on earlier versions, but it should be OK for later ones. An alias with suitable caveats sounds reasonable. |
Any idea what it is that's incompatible with earlier versions of SQL Server - we have to support SQL Server 2005 (but typically develop against 2008 or later). We're not currently using the paging functionality, but it might be useful for later as we move more of the codebase to ElSql. |
No idea I'm afraid. If you can test the various bits of SQL on 2005 that would be great as we can then document the issues. |
I think it was the new date and time types. |
We're going to try and spin up a test SQL Server 2005 instance so that we can test the paging SQL. |
ElSql doesn't know about date and time types so that must have been OpenGamma. So it might be that the OpenGamma restriction caused an unecessary restriction in ElSql. Basically, the database specific stuff is defined in the config class. Providing these elements work in SQL server 2005 you should be fine. The rest of ElSql is standard SQL or just passing your SQL around. Looking forward to hearing your results! |
Yes, actually, the date and times thing must have been a deeper OG code restriction. From what I can tell from looking at the code, the 2008 pagination should work on 2005. I guess your experiment will find out one way or the other definitively. |
Is there anything in the SqlServer2008 configuration that is known to be specific to the 2008 version of SQL Server?
If not, it would be preferable to have a generic SqlServer config, and SqlServer2008 could be an alias for that (for anyone who is already using it).
I'm willing to put in a pull request if it sounds like a reasonable change.
Thanks,
Andrew
The text was updated successfully, but these errors were encountered: