[GitHub] [airflow] bjoernpollex-sc commented on issue #4743: [AIRFLOW-3871] render Operators template fields recursively

2019-09-18 Thread GitBox
bjoernpollex-sc commented on issue #4743: [AIRFLOW-3871] render Operators template fields recursively URL: https://github.com/apache/airflow/pull/4743#issuecomment-532723570 Same here, this is really awesome, can't wait for the next release!

[GitHub] [airflow] bjoernpollex-sc commented on issue #4743: [AIRFLOW-3871] render Operators template fields recursively

2019-05-31 Thread GitBox
bjoernpollex-sc commented on issue #4743: [AIRFLOW-3871] render Operators template fields recursively URL: https://github.com/apache/airflow/pull/4743#issuecomment-497733970 @galak75 Thanks again for the work, hoping to see this in soon! I was wondering though, how does this work with the

[GitHub] [airflow] bjoernpollex-sc commented on issue #4743: [AIRFLOW-3871] render Operators template fields recursively

2019-03-15 Thread GitBox
bjoernpollex-sc commented on issue #4743: [AIRFLOW-3871] render Operators template fields recursively URL: https://github.com/apache/airflow/pull/4743#issuecomment-473192025 @ashb Suppose you have a `GCSFileTransformOperator` that applies any given transformation. The problem is that

[GitHub] [airflow] bjoernpollex-sc commented on issue #4743: [AIRFLOW-3871] render Operators template fields recursively

2019-03-14 Thread GitBox
bjoernpollex-sc commented on issue #4743: [AIRFLOW-3871] render Operators template fields recursively URL: https://github.com/apache/airflow/pull/4743#issuecomment-472735428 @galak75 Thanks, this looks very good. The implementation seems fine to me, and the tests seem to cover all the

[GitHub] [airflow] bjoernpollex-sc commented on issue #4743: [AIRFLOW-3871] render Operators template fields recursively

2019-03-13 Thread GitBox
bjoernpollex-sc commented on issue #4743: [AIRFLOW-3871] render Operators template fields recursively URL: https://github.com/apache/airflow/pull/4743#issuecomment-472317330 @galak75 Good points! Responses below: 1. Airflow templating cannot always be used. For example, we have an

[GitHub] [airflow] bjoernpollex-sc commented on issue #4743: [AIRFLOW-3871] render Operators template fields recursively

2019-03-12 Thread GitBox
bjoernpollex-sc commented on issue #4743: [AIRFLOW-3871] render Operators template fields recursively URL: https://github.com/apache/airflow/pull/4743#issuecomment-471892449 @galak75 Btw., not sure if you've seen this, but I've also [commented on the ticket][1] regarding the general