Lee-W commented on code in PR #43773:
URL: https://github.com/apache/airflow/pull/43773#discussion_r1843370667
##########
newsfragments/43773.significant.rst:
##########
Review Comment:
Yep, sounds good. Will update ti
##########
providers/src/airflow/providers/amazon/aws/assets/s3.py:
##########
@@ -19,14 +19,30 @@
from typing import TYPE_CHECKING
from airflow.providers.amazon.aws.hooks.s3 import S3Hook
-from airflow.providers.common.compat.assets import Asset
if TYPE_CHECKING:
from urllib.parse import SplitResult
+ from airflow.providers.common.compat.assets import Asset
from airflow.providers.common.compat.openlineage.facet import (
Dataset as OpenLineageDataset,
)
+else:
+ # TODO: Remove this try-exception block after bumpping common provider to
1.3.0
Review Comment:
will fix all the typos
--
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]
