-
Notifications
You must be signed in to change notification settings - Fork 562
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 for issue 617 #623
Fix for issue 617 #623
Conversation
Merging updates.
merge main into fork
Merge with master
Merge with master
merge with master
Bugfix tvp type mismatch
Looks good. Small change request - could you change the table to "t1". Tests use t0-t2 which are automatically dropped before each test. (Also, I'm not sure why it is using |
Merge with master
@mkleehammer I renamed the table, as for why it's |
Fixing the insertion of high unicode characters, bug caused by SQLPutData being given the number of characters, rather then code points, which causes problems for characters are more code points then usual