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/08 21:13:21 UTC

[GitHub] [airflow] ephraimbuddy commented on a change in pull request #19572: Simplify KubernetesPodOperator

ephraimbuddy commented on a change in pull request #19572:
URL: https://github.com/apache/airflow/pull/19572#discussion_r765245589



##########
File path: airflow/providers/cncf/kubernetes/operators/kubernetes_pod.py
##########
@@ -15,17 +15,26 @@
 # specific language governing permissions and limitations
 # under the License.
 """Executes task in a Kubernetes POD"""
+import json
 import re
 import warnings
-from typing import TYPE_CHECKING, Any, Dict, Iterable, List, Optional, Tuple, Type
+from contextlib import AbstractContextManager
+from typing import TYPE_CHECKING, Any, Dict, Iterable, List, Optional
 
 from kubernetes.client import CoreV1Api, models as k8s
 
+from airflow.providers.cncf.kubernetes.utils.pod_launcher import PodLaunchFailedException, PodStatus
+
 try:
     import airflow.utils.yaml as yaml
 except ImportError:
     import yaml
 
+try:
+    from functools import cached_property
+except ImportError:
+    from cached_property import cached_property
+

Review comment:
       ```suggestion
   from airflow.compat.functools import cached_property
   
   ```




-- 
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