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