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
Syntax error when creating an index #287
Comments
Hi, I need more information to be able to help you here. What version of pgloader are you using ( |
Running from source, commit b78bb6d. It's ms sql. |
As seen in #287 the previous decision to force quoting to :none is wrong, because index names in MS SQL source database might contain spaces, and then need to be quoted. Let's see what happens if we do it the usual way for MS SQL too, and allow users to control the quoting behaviour of pgloader here.
So I just added what I think should fix the problem, with an option for you to control pgloader's behavior. In the Should you confirm it works with that change, I will then cover that in the documentations and close the issue. Thanks for reporting this bug! |
When parsing table names in the target URI, we are careful of splitting the table and schema name and store them into a cons in that case. Not all sources methods got the memo, clean that up. See #182 and #186, a pull request I am now going to be able to accept. Also see #287 that should be helped by being able to apply #186.
See also #186 that has been now properly merged into pgloader. |
Oh oops. Clicked "close" inadvertently. Please re-open if you still have the bug! |
I think maybe the index name should be quoted?
The text was updated successfully, but these errors were encountered: