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

Expose SQL Types in Cursor #621

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

celestialorb
Copy link

This is a follow-up PR for issue #167, which has had a PR submitted already (#168). When testing this modified pyodbc package locally I found that the coldescription attribute of the cursor was not set properly for procedures that return more than one result set.

@mkleehammer
Copy link
Owner

Very nice and clean.

I know it seems analogous to Cursor.description, but I am concerned about adding any more processing for something that will be rarely used. When needed it is important, but I think it won't be needed most of the time. What do you think about turning it into a function instead?

It might also be nice to figure out if we can easily make the return type a list of namedtuples. Then we could document that attributes may be added to the tuple in the future, so use the names instead of tuple unpacking.

@mkleehammer
Copy link
Owner

I've put this on hold for v5 due to the suggestions I made above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants