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/03/02 18:52:13 UTC

[GitHub] [airflow] turbaszek commented on a change in pull request #14554: Remove unnecessary WARNINGs from BeamHook

turbaszek commented on a change in pull request #14554:
URL: https://github.com/apache/airflow/pull/14554#discussion_r585823372



##########
File path: airflow/providers/apache/beam/hooks/beam.py
##########
@@ -25,7 +25,7 @@
 from typing import Callable, List, Optional
 
 from airflow.exceptions import AirflowException
-from airflow.hooks.base_hook import BaseHook
+from airflow.hooks.base import BaseHook

Review comment:
       Yes we do refactor, all changes are here I think:
   https://github.com/apache/airflow/blob/3064bf04429f86ff2b527704603ef3ca9b9fe22f/dev/provider_packages/copy_provider_package_sources.py#L82




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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org