mik-laj commented on a change in pull request #8011: Rename CloudBaseHook to 
GoogleBaseHook and move it to google.common
URL: https://github.com/apache/airflow/pull/8011#discussion_r400562737
 
 

 ##########
 File path: airflow/providers/google/cloud/hooks/automl.py
 ##########
 @@ -29,10 +29,10 @@
     ImageObjectDetectionModelDeploymentMetadata, InputConfig, Model, 
Operation, PredictResponse, TableSpec,
 )
 
-from airflow.providers.google.cloud.hooks.base import CloudBaseHook
+from airflow.providers.google.common.hooks.base import GoogleBaseHook
 
 Review comment:
   ```suggestion
   from airflow.providers.google.common.hooks.base_google import GoogleBaseHook
   ```
   We agreed to such a convention in the case of AWS. In the case of Google, we 
should also use it.  I will add a commit to this branch, which corrects it, and 
then from this branch I will start new PR, which will correct other errors.

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to