-
Notifications
You must be signed in to change notification settings - Fork 109
Add support for user-defined data types (Sybase only) #325
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
Conversation
Related issue: tds-fdw#323
|
Can one of the admins verify this patch? |
|
Test this, please |
|
Test PASSed. |
|
Let's test it again, and if so, maybe it can be merged. Test this, please. |
|
Test PASSed. |
|
Hi @vitalca , Thanks for the PR! Sorry for letting this one sit for a while. What do you think about the following changes for this?:
|
|
Hi @GeoffMontee, That makes sense. Thanks for the suggestion! |
|
In which version will this feature be implemented? |
|
@lcnogueirajr, I think we first need @vitalca addressing @GeoffMontee comment. Now that 2.0.5 is out, it's too late for it, but nothing prevents a 2.0.6 after this is fixed and merged. |
|
New PR has been created |
|
Thank you, @vitalca! |
Related issue: #323