[GitHub] [airflow] ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization

2019-12-06 Thread GitBox
ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization URL: https://github.com/apache/airflow/pull/6715#discussion_r354901371 ## File path: airflow/serialization/serialized_objects.py ## @@ -354,6 +370,65 @

[GitHub] [airflow] ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization

2019-12-06 Thread GitBox
ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization URL: https://github.com/apache/airflow/pull/6715#discussion_r354900317 ## File path: airflow/serialization/serialized_objects.py ## @@ -354,6 +370,65 @

[GitHub] [airflow] ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization

2019-12-06 Thread GitBox
ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization URL: https://github.com/apache/airflow/pull/6715#discussion_r354900317 ## File path: airflow/serialization/serialized_objects.py ## @@ -354,6 +370,65 @

[GitHub] [airflow] ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization

2019-12-06 Thread GitBox
ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization URL: https://github.com/apache/airflow/pull/6715#discussion_r354899097 ## File path: airflow/serialization/serialized_objects.py ## @@ -354,6 +370,65 @

[GitHub] [airflow] ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization

2019-12-05 Thread GitBox
ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization URL: https://github.com/apache/airflow/pull/6715#discussion_r354555281 ## File path: tests/serialization/test_dag_serialization.py ## @@ -381,6 +362,17

[GitHub] [airflow] ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization

2019-12-05 Thread GitBox
ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization URL: https://github.com/apache/airflow/pull/6715#discussion_r354552987 ## File path: airflow/serialization/serialized_objects.py ## @@ -354,6 +371,88 @

[GitHub] [airflow] ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization

2019-12-05 Thread GitBox
ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization URL: https://github.com/apache/airflow/pull/6715#discussion_r354553355 ## File path: airflow/utils/tests.py ## @@ -71,15 +75,70 @@ class Dummy3TestOper

[GitHub] [airflow] ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization

2019-12-05 Thread GitBox
ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization URL: https://github.com/apache/airflow/pull/6715#discussion_r354553508 ## File path: airflow/utils/tests.py ## @@ -71,15 +75,70 @@ class Dummy3TestOper

[GitHub] [airflow] ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization

2019-12-05 Thread GitBox
ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization URL: https://github.com/apache/airflow/pull/6715#discussion_r354280914 ## File path: airflow/serialization/serialized_objects.py ## @@ -350,6 +368,53 @

[GitHub] [airflow] ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization

2019-12-05 Thread GitBox
ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization URL: https://github.com/apache/airflow/pull/6715#discussion_r354279512 ## File path: tests/gcp/operators/test_bigquery.py ## @@ -432,6 +433,93 @@ def t

[GitHub] [airflow] ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization

2019-12-05 Thread GitBox
ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization URL: https://github.com/apache/airflow/pull/6715#discussion_r354244193 ## File path: airflow/models/baseoperator.py ## @@ -1081,21 +1082,22 @@ def get_

[GitHub] [airflow] ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization

2019-12-04 Thread GitBox
ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization URL: https://github.com/apache/airflow/pull/6715#discussion_r353691980 ## File path: tests/serialization/test_dag_serialization.py ## @@ -381,6 +362,49

[GitHub] [airflow] ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization

2019-12-04 Thread GitBox
ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization URL: https://github.com/apache/airflow/pull/6715#discussion_r353690550 ## File path: airflow/serialization/serialized_objects.py ## @@ -202,7 +203,11 @

[GitHub] [airflow] ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization

2019-12-04 Thread GitBox
ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization URL: https://github.com/apache/airflow/pull/6715#discussion_r353686706 ## File path: airflow/contrib/operators/qubole_operator.py ## @@ -240,3 +241,9 @

[GitHub] [airflow] ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization

2019-12-04 Thread GitBox
ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization URL: https://github.com/apache/airflow/pull/6715#discussion_r353675344 ## File path: airflow/serialization/serialized_baseoperator.py ## @@ -124,3 +142

[GitHub] [airflow] ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization

2019-12-03 Thread GitBox
ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization URL: https://github.com/apache/airflow/pull/6715#discussion_r353114354 ## File path: airflow/serialization/serialized_baseoperator.py ## @@ -124,3 +142

[GitHub] [airflow] ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization

2019-12-03 Thread GitBox
ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization URL: https://github.com/apache/airflow/pull/6715#discussion_r353113943 ## File path: airflow/serialization/serialized_baseoperator.py ## @@ -124,3 +142

[GitHub] [airflow] ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization

2019-12-03 Thread GitBox
ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization URL: https://github.com/apache/airflow/pull/6715#discussion_r353112329 ## File path: airflow/serialization/serialized_baseoperator.py ## @@ -89,7 +96,1

[GitHub] [airflow] ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization

2019-12-03 Thread GitBox
ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization URL: https://github.com/apache/airflow/pull/6715#discussion_r353111076 ## File path: airflow/contrib/operators/qubole_operator.py ## @@ -240,3 +241,9 @

[GitHub] [airflow] ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization

2019-12-03 Thread GitBox
ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization URL: https://github.com/apache/airflow/pull/6715#discussion_r353109104 ## File path: airflow/serialization/base_serialization.py ## @@ -216,7 +216,11 @

[GitHub] [airflow] ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization

2019-12-03 Thread GitBox
ashb commented on a change in pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization URL: https://github.com/apache/airflow/pull/6715#discussion_r353107800 ## File path: airflow/contrib/operators/qubole_operator.py ## @@ -240,3 +241,9 @