{"action":"created","issue":{"url":"https://api.github.com/repos/apache/incubator-airflow/issues/1745","repository_url":"https://api.github.com/repos/apache/incubator-airflow","labels_url":"https://api.github.com/repos/apache/incubator-airflow/issues/1745/labels{/name}","comments_url":"https://api.github.com/repos/apache/incubator-airflow/issues/1745/comments","events_url":"https://api.github.com/repos/apache/incubator-airflow/issues/1745/events","html_url":"https://github.com/apache/incubator-airflow/pull/1745","id":171769300,"number":1745,"title":"[Airflow-409] Polite Logging Configuration","user":{"login":"mtdewulf","id":3631601,"avatar_url":"https://avatars.githubusercontent.com/u/3631601?v=3","gravatar_id":"","url":"https://api.github.com/users/mtdewulf","html_url":"https://github.com/mtdewulf","followers_url":"https://api.github.com/users/mtdewulf/followers","following_url":"https://api.github.com/users/mtdewulf/following{/other_user}","gists_url":"https://api.github.com/users/mtdewulf/gists{/gist_id}","starred_url":"https://api.github.com/users/mtdewulf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mtdewulf/subscriptions","organizations_url":"https://api.github.com/users/mtdewulf/orgs","repos_url":"https://api.github.com/users/mtdewulf/repos","events_url":"https://api.github.com/users/mtdewulf/events{/privacy}","received_events_url":"https://api.github.com/users/mtdewulf/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2016-08-17T21:51:26Z","updated_at":"2016-09-09T16:53:22Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/incubator-airflow/pulls/1745","html_url":"https://github.com/apache/incubator-airflow/pull/1745","diff_url":"https://github.com/apache/incubator-airflow/pull/1745.diff","patch_url":"https://github.com/apache/incubator-airflow/pull/1745.patch"},"body":"Dear Airflow Maintainers,\r\n\r\nPlease accept this PR that addresses the following issues:\r\n- https://issues.apache.org/jira/browse/AIRFLOW-409\r\n\r\nThis PR avoids the pattern of:\r\n\r\n```\r\nlogging.getLogger().handlers = []  # Blow away existing root handlers.\r\nlogging.basicConfig(...)  # Install a simple stream handler to standard error/out.\r\n```\r\n\r\nAnd instead calls `addHandler` as appropriate.  This also avoids changing the root logger's log level and only modifies handler log levels.\r\n\r\nTesting Done:\r\n- Added unit tests.\r\n- Tested out CLI commands to verify logging still functioned."},"comment":{"url":"https://api.github.com/repos/apache/incubator-airflow/issues/comments/245972158","html_url":"https://github.com/apache/incubator-airflow/pull/1745#issuecomment-245972158","issue_url":"https://api.github.com/repos/apache/incubator-airflow/issues/1745","id":245972158,"user":{"login":"criccomini","id":167569,"avatar_url":"https://avatars.githubusercontent.com/u/167569?v=3","gravatar_id":"","url":"https://api.github.com/users/criccomini","html_url":"https://github.com/criccomini","followers_url":"https://api.github.com/users/criccomini/followers","following_url":"https://api.github.com/users/criccomini/following{/other_user}","gists_url":"https://api.github.com/users/criccomini/gists{/gist_id}","starred_url":"https://api.github.com/users/criccomini/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/criccomini/subscriptions","organizations_url":"https://api.github.com/users/criccomini/orgs","repos_url":"https://api.github.com/users/criccomini/repos","events_url":"https://api.github.com/users/criccomini/events{/privacy}","received_events_url":"https://api.github.com/users/criccomini/received_events","type":"User","site_admin":false},"created_at":"2016-09-09T16:53:22Z","updated_at":"2016-09-09T16:53:22Z","body":"LGTM, but I'm also no expert on Python logging. Perhaps @bolkedebruin @mistercrunch @aoen should have a look"}}
{"actor":{"display_login":"criccomini"}}