[GitHub] [airflow] serkef commented on issue #5557: [AIRFLOW-4928] Move all config parses to class properties

2019-09-18 Thread GitBox
serkef commented on issue #5557: [AIRFLOW-4928] Move all config parses to class properties URL: https://github.com/apache/airflow/pull/5557#issuecomment-532877001 @ashb It's good to go This is an automated message from the Ap

[GitHub] [airflow] serkef commented on issue #5557: [AIRFLOW-4928] Move all config parses to class properties

2019-08-25 Thread GitBox
serkef commented on issue #5557: [AIRFLOW-4928] Move all config parses to class properties URL: https://github.com/apache/airflow/pull/5557#issuecomment-524633452 @ashb Fixed your comments and rebased. Please take another look. --

[GitHub] [airflow] serkef commented on issue #5557: [AIRFLOW-4928] Move all config parses to class properties

2019-07-13 Thread GitBox
serkef commented on issue #5557: [AIRFLOW-4928] Move all config parses to class properties URL: https://github.com/apache/airflow/pull/5557#issuecomment-511096329 @ashb I don't have a benchmark. The default arguments are evaluated in parsing, but still we don't need to evaluate multiple ti