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

cur will be undefined if ysql.connect fails #4

Open
glennra opened this issue Feb 15, 2018 · 3 comments
Open

cur will be undefined if ysql.connect fails #4

glennra opened this issue Feb 15, 2018 · 3 comments

Comments

@glennra
Copy link

glennra commented Feb 15, 2018

cur.close()

@glennra
Copy link
Author

glennra commented Feb 15, 2018

not sure of the best way to fix this but maybe:

if 'cur' in locals():
    cur.close()

@rsvp
Copy link
Owner

rsvp commented Feb 15, 2018

@glennra hi Glen, the fallbacks will be corrected when yserial
gets rewritten for python3 using better flow constructs.

Thanks for your attention in the matter.

Other than the current issue, are there any critical fails
when you run the code against python2 ?

@glennra
Copy link
Author

glennra commented Feb 15, 2018

Yes, in 0.60 there was an issue with error reporting that I fixed in my copy. I will have to look it up. I'll create a new issue if the problem still exists.

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

No branches or pull requests

2 participants