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

different collations fix #3786

Merged
merged 1 commit into from
Mar 7, 2018
Merged

different collations fix #3786

merged 1 commit into from
Mar 7, 2018

Conversation

dilshatm
Copy link
Contributor

@dilshatm dilshatm commented Feb 12, 2018

Required for all PRs:

  • Signed CLA.
  • Associated README.md updated.
  • Has appropriate unit tests.

@danielnelson
Copy link
Contributor

@m82labs Any chance we would need something similar on the v2 query type?

@danielnelson danielnelson added this to the 1.5.3 milestone Feb 16, 2018
@danielnelson danielnelson added the fix pr to fix corresponding bug label Feb 16, 2018
@m82labs
Copy link
Contributor

m82labs commented Mar 2, 2018

I would need to better understand what the reasoning was for adding this. @dilshatm what was the reason for this change, what errors were you seeing?

@dilshatm
Copy link
Contributor Author

dilshatm commented Mar 6, 2018

Collation of the database in connectionstring is Cyrillic_General_CI_AS. Collation of tempdb is SQL_Latin1_General_CP1_CI_AS.

For example, running sqlWaitStatsCategorized in first database caused 2 error message

Msg 468, Level 16, State 9, Line 270
Cannot resolve the collation conflict between "Cyrillic_General_CI_AS" and "SQL_Latin1_General_CP1_CI_AS" in the equal to operation.
Msg 468, Level 16, State 9, Line 286
Cannot resolve the collation conflict between "Cyrillic_General_CI_AS" and "SQL_Latin1_General_CP1_CI_AS" in the equal to operation.

For version2 itsn't actual, because there is no use of #temp tables.

@danielnelson danielnelson merged commit a6d366f into influxdata:master Mar 7, 2018
danielnelson pushed a commit that referenced this pull request Mar 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix pr to fix corresponding bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants