BUG/TST: catch socket.error in py2/3.2 and ConnectionError in py3.3

[cpcloud]

closes #3982.

[cpcloud]

this passes travis but that doesn't really tell us that much...since the errors from #3982 are occurring intermittently...

[cpcloud]

also it skips when i remove my connection...but i'm still not sure if that's enough

[cpcloud]

strange that it only happens on 3.2...trying to think of what itcould be @jreback @jtratner any ideas?
https://travis-ci.org/cpcloud/pandas/jobs/8322345

[jtratner]

Is socket.error a subclass of IOError in 3.2? Wasn't clear on the
resolution to this previously...

[cpcloud]

yep it is.

[cpcloud]

it's possible that this might be the problem and the solution

[cpcloud]

sigh probably a simple context manager will fix this....the resp is not being closed...same for google

[jtratner]

I can write something up this week, do you want me to take it on?

[cpcloud]

no it's fine i can do it, won't take that long. i'll write something on the wiki too showing how to use context managers with resources that don't implement the context manager protocol.

while it may not be too big of a deal to not close a file (or close url resource) when one is working in the shell, it's quite another to leave resources open in library code, especially since python makes it so easy to automatically close a file.

[jtratner]

can you use urlopen in a with statement? If so, that makes it incredibly
simple....

On Sat, Jun 22, 2013 at 11:39 PM, Phillip Cloud [email protected]:

no it's fine i can do it, won't take that long. i'll write something on
the wiki too showing how to use context managers with resources that don't
implement the context manager protocol.

while it may not be too big of a deal to not close a file (or close url
resource) when one is working in the shell, it's quite another to leave
resources open in library code, especially since python makes it _so_easy to automatically close a file.

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/3985#issuecomment-19868544
.

[cpcloud]

Yep but in Python 2 you must use for example, with contextlib.closing(urlopen(url)) as response since url objects don't implement the context manager protocol in Python 2 in Python 3 they work fine with a regular with statement

[jtratner]

good to know - learned something today :)

On Sun, Jun 23, 2013 at 9:29 AM, Phillip Cloud [email protected]:

Yep but in Python 2 you must use for example, with
contextlib.closing(urlopen(url)) as response since url objects don't
implement the context manager protocol in Python 2 in Python 3 they work
fine with a regular with statement

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/3985#issuecomment-19873954
.

[jreback]

this fix the problem?

[cpcloud]

close in favor of #4002 .

[jreback]

since you moved your data fixes to 0.13.....any chance of reviving fixing these network test issues? or too much work?

[cpcloud]

i could just peel off the commit from the clean up and use that

[jreback]

maybe do that these netwok tests failing too much

[cpcloud]

let me get rid of some of these merge commits

[cpcloud]

whoops that zf context won't pass hold on

[cpcloud]

no wonder there are all these reset connection errors there are a ton of sockets to yahoo finance left open

[cpcloud]

will merge after passing...just added docs