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

[information_schema] case issue #6

Closed
aaroncalderon opened this issue Aug 22, 2017 · 1 comment
Closed

[information_schema] case issue #6

aaroncalderon opened this issue Aug 22, 2017 · 1 comment

Comments

@aaroncalderon
Copy link
Contributor

I was trying to run sql-ts against a ms sql 2008 server, but I was not able to go past the adapter query where it gets all the table info for a database Adapters/mssql.ts#L6 and #L14.

I successfully passed through that by changing the case to upper case:

#L6 const query = db('INFORMATION_SCHEMA.TABLES') and #L14 const query = db('INFORMATION_SCHEMA.COLUMNS')

I do not now how this change affects other versions of SQL server.

@rmp135
Copy link
Owner

rmp135 commented Aug 23, 2017

Cheers. I've changed the case to uppercase and pushed as 0.2.4. I think this could be a collation issue with case sensitive table names. As far as I'm aware all versions of MSSQL have uppercase system names so this shouldn't cause any issues.

@rmp135 rmp135 closed this as completed Aug 23, 2017
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