pagure

#860 Improve logging and attempt to fix `too many open files` in the EV server
Merged by pingou. Opened by pingou.
improve_ev_server  into  master

Download 860.patch
no initial comment

Pull-Request has been rebased

FYI, I think what you want here is just:

log.exception("Failure in handle_client")

log.exception automatically logs the current exception in play.. so you passing the exception to it is redundant. You instead want to pass some supplemental information to log.exception since it already has access to err automatically. These two log.info and log.exception statements can be collapsed into one.

Same down here, I think.

Pull-Request has been updated

LGTM!

Thanks! :)

Pull-Request has been merged by pingou
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.