[GitHub] [airflow] o-nikolas commented on a diff in pull request #28827: Add option to wait for completion on the EmrCreateJobFlowOperator

2023-01-24 Thread via GitHub
o-nikolas commented on code in PR #28827: URL: https://github.com/apache/airflow/pull/28827#discussion_r1086224824 ## airflow/providers/amazon/aws/operators/emr.py: ## @@ -538,42 +544,76 @@ def __init__( emr_conn_id: str | None = "emr_default",

[GitHub] [airflow] o-nikolas commented on a diff in pull request #28827: Add option to wait for completion on the EmrCreateJobFlowOperator

2023-01-10 Thread GitBox
o-nikolas commented on code in PR #28827: URL: https://github.com/apache/airflow/pull/28827#discussion_r1066407359 ## airflow/providers/amazon/aws/utils/waiter.py: ## @@ -60,14 +60,20 @@ def waiter( break if state in failure_states: raise

[GitHub] [airflow] o-nikolas commented on a diff in pull request #28827: Add option to wait for completion on the EmrCreateJobFlowOperator

2023-01-10 Thread GitBox
o-nikolas commented on code in PR #28827: URL: https://github.com/apache/airflow/pull/28827#discussion_r1066357512 ## airflow/providers/amazon/aws/operators/emr.py: ## @@ -538,42 +544,76 @@ def __init__( emr_conn_id: str | None = "emr_default",

[GitHub] [airflow] o-nikolas commented on a diff in pull request #28827: Add option to wait for completion on the EmrCreateJobFlowOperator

2023-01-10 Thread GitBox
o-nikolas commented on code in PR #28827: URL: https://github.com/apache/airflow/pull/28827#discussion_r1066269743 ## airflow/providers/amazon/aws/operators/emr.py: ## @@ -538,42 +544,76 @@ def __init__( emr_conn_id: str | None = "emr_default",