mobuchowski commented on code in PR #45260:
URL: https://github.com/apache/airflow/pull/45260#discussion_r1928442661


##########
providers/src/airflow/providers/openlineage/extractors/manager.py:
##########
@@ -36,8 +36,8 @@
 if TYPE_CHECKING:
     from openlineage.client.event_v2 import Dataset
 
-    from airflow.lineage.entities import Table
     from airflow.models import Operator
+    from airflow.providers.common.compat.lineage.entities import Table

Review Comment:
   @uranusjr those classes haven't changed in a while, the compat classes will 
work too.
   
   Only thing necessary IMO is to bump the required version of 
`apache-airflow-providers-common-compat` in OpenLineage provider's 
provider.yaml - those would work on the old versions,



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

Reply via email to