You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/12/30 17:08:50 UTC

[GitHub] [airflow] dstandish commented on a change in pull request #20575: Delete pods by default in KubernetesPodOperator

dstandish commented on a change in pull request #20575:
URL: https://github.com/apache/airflow/pull/20575#discussion_r776807136



##########
File path: airflow/providers/amazon/aws/operators/eks.py
##########
@@ -18,12 +18,13 @@
 """This module contains Amazon EKS operators."""
 import warnings
 from time import sleep
-from typing import Dict, Iterable, List, Optional
+from typing import Dict, Iterable, List, Optional, Union
 
 from airflow import AirflowException
 from airflow.models import BaseOperator
 from airflow.providers.amazon.aws.hooks.eks import ClusterStates, EksHook, FargateProfileStates
 from airflow.providers.cncf.kubernetes.operators.kubernetes_pod import KubernetesPodOperator
+from airflow.utils.types import NOTSET, ArgNotSet

Review comment:
       dangit, min version strikes again!




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org