search

Yelp / zygote

A Python HTTP process management utility.
http://opensource.yelp.com/
Apache License 2.0
40 stars 15 forks source link

handle initialization errors better #17

Closed Roguelazer closed 12 years ago

Roguelazer commented 12 years ago

Logs a lot more things, and gives up if the initialization of the ZygoteWorker fails rather than infinite-looping. The ZygoteIOLoop stuff is pretty janky, but tornado is very insistent about logging to the root logger unless you give it a solid thwack on the nose. As I put in the XXX comment, I'd like to do something similar for RequestHandler, but I'm not sure where a good place to do so is.

Roguelazer commented 12 years ago

Note: this should close issue #15.

Roguelazer commented 12 years ago

jared found a case where the atexit handler could be called without the ioloop existing, so I now do a getattr to prevent a stupid bug.

Roguelazer commented 12 years ago

Anything else, Eskil?

eskil commented 12 years ago

All good!