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

column database type names #3

Closed
wants to merge 1 commit into from
Closed

column database type names #3

wants to merge 1 commit into from

Conversation

jpalawaga
Copy link
Member

@jpalawaga jpalawaga commented Oct 19, 2021

Column type (type name?) support is an excellent thing to have. Looking around github, I see there are a few different implementations of this. I'm not sure which is the ideal pick to merge, this one doesn't seem too bad as it just surfaces the SQL C types, which is a pretty reasonable first step.

I'll look around before merging this.

@jpalawaga
Copy link
Member Author

OG Pull request: alexbrainman/odbc#114

@jpalawaga jpalawaga closed this in 9a7df19 Oct 20, 2021
@jpalawaga jpalawaga deleted the columnnames branch October 20, 2021 20:56
jpalawaga added a commit that referenced this pull request Nov 1, 2021
I think ideally we'd be able to get the name from the driver rather than
defaulting to the c-api's name, but I'm not sure how possible that is at
this time.

This seems like a fine intermediary for now.

Closes #3

Co-authored-by:  Roman TSovanyan <covrom@gmail.com>
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

Successfully merging this pull request may close these issues.

2 participants