Bug #5016
closed
- Assigned To set to Peter Amstutz
- Subject changed from Should be able to use ^C to interrupt arv-ws. to [SDKs] Should be able to use ^C to interrupt arv-ws.
691d642 does the job, but it can be surprisingly sluggish. Here's a transcript where I hit ^C as soon as I saw the fallback notices:
brinstar % arv-ws; date
2015-01-21 10:12:03 arvados.events[21214] WARNING: Got exception _ssl.c:331: No root certificates specified for verification of other-side certificates. trying to connect to websockets at wss://ws.qr1hi.arvadosapi.com/websocket
2015-01-21 10:12:03 arvados.events[21214] WARNING: Websockets not available, falling back to log table polling
^CWed Jan 21 10:12:16 EST 2015
Taking about 10-13 seconds is what I typically saw, independently of how I set the poll interval. If there's a way to make it snappier, that seems like it'd be nice, but if not, this is good to merge. Thanks.
Okay, I see now that what's going on in my last comment is that the thread is blocking on the first large API response of logs, and that ^C reacts more like expected after that. So that leaves me less concerned and I'm happy to see the branch merged as-is.
- Status changed from In Progress to Resolved
- % Done changed from 0 to 100
Applied in changeset arvados|commit:ffcfb73e6a5dfd626b1d523accc42e7a6af5e8ed.
Also available in: Atom
PDF