wojtek2kdev commented on issue #8471:
URL: https://github.com/apache/airflow/issues/8471#issuecomment-626630255


   @mik-laj I think we can. We just only need to choose calendar service and 
prepare desired query. So should we create Google API to GCS transfer operator? 
Will it be sufficient for case described in issue + calendar hook for events 
create/update/remove? 
   
   Additionaly, I have a few notes, one - I know that's more art than science, 
but I think it would be better if operators was organize by type, use-case 
packages e.g having s3_to_gcs or bigquery_to_gcs or bigquery_to_mysql and so 
on, those should be in e.g **operators.transfer** package, operators which only 
executes api CRUD operations should be in **operators.api**, etc. I think we 
can extract more operation types, and create packages to keep order, instead of 
having so many different operators in single folder. Just proposition.
   
   Two - why we create hooks which only wraps API calls? For example 
[sheets.py](https://github.com/apache/airflow/blob/5f3774aebdce8fe7bdcc478ea0b05d95c37c9440/airflow/providers/google/suite/hooks/sheets.py).
 Also, why we must implement get_conn method in all hooks/operators using 
google services, instead of inherit that or make some smart decorator?


----------------------------------------------------------------
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:
[email protected]


Reply via email to