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

Fix an issue of adding OFFSET keyword for legacy SQL Server versions #218

Merged
merged 1 commit into from Jan 10, 2023

Conversation

adelkhayata76
Copy link
Contributor

I was getting the following exception when trying to access legacy SQL Server 2008 R2 database: pyodbc.ProgrammingError:

('42000', "[42000] [Microsoft][ODBC Driver 17 for SQL Server][SQL Server]Incorrect syntax near 'OFFSET'. (102) (SQLExecDirectW)")

I only added one more condition before adding the OFFSET keyword.

I was getting the following exception when trying to access legacy SQL Server 2008 R2 database:
pyodbc.ProgrammingError: ('42000', "[42000] [Microsoft][ODBC Driver 17 for SQL Server][SQL Server]Incorrect syntax near 'OFFSET'. (102) (SQLExecDirectW)")

I only added one more condition before adding the OFFSET keyword.
@adelkhayata76 adelkhayata76 changed the title Fix an issue of adding OFFSET keyword for legacy SQL Server vresions Fix an issue of adding OFFSET keyword for legacy SQL Server versions Dec 28, 2022
@mShan0 mShan0 merged commit 17bb0c8 into microsoft:dev Jan 10, 2023
@mShan0
Copy link
Contributor

mShan0 commented Jan 10, 2023

Thanks for the PR!

@adelkhayata76
Copy link
Contributor Author

Hey,

Sorry to ask, but when do we expect a new version?

@mShan0
Copy link
Contributor

mShan0 commented Feb 15, 2023

We don't have a set release date yet but it should be around the time that Django 4.2 is released

@adelkhayata76
Copy link
Contributor Author

We don't have a set release date yet but it should be around the time that Django 4.2 is released

Hey, Django 4.2 is released, is it possible to publish a new version now?

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

Successfully merging this pull request may close these issues.

None yet

2 participants