kaxil commented on code in PR #56793: URL: https://github.com/apache/airflow/pull/56793#discussion_r2451940072
########## providers/google/src/airflow/providers/google/cloud/links/base.py: ########## @@ -20,17 +20,8 @@ from typing import TYPE_CHECKING, ClassVar from urllib.parse import urlparse -from airflow.providers.google.version_compat import ( - AIRFLOW_V_3_0_PLUS, - BaseOperator, - BaseOperatorLink, - BaseSensorOperator, -) - -if AIRFLOW_V_3_0_PLUS: - from airflow.sdk.execution_time.xcom import XCom -else: - from airflow.models.xcom import XCom # type: ignore[no-redef] +from airflow.providers.common.compat.lazy_compat import BaseOperatorLink, BaseSensorOperator, XCom Review Comment: No, BaseOperator is special cased for 3.1 due to xcom_push bug -- there should be a comment about it in version_compat file -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
