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 2020/08/25 12:59:04 UTC

[GitHub] [airflow] mschickensoup commented on issue #8286: Cloud Memorystore for Memcached operators

mschickensoup commented on issue #8286:
URL: https://github.com/apache/airflow/issues/8286#issuecomment-680008894


   Hello my @tanjinP 
   
   I would like to thank you for this integration. This will make it much easier for everyone to use this service.
   
   I've talked to some people at my team and there are a few comments:
   
   - Have you thought to create the CloudMemorystoreMemcachedGetInstanceOperator operator? This would make it easier to link several operators together.
   - In the file tests/providers/google/cloud/hooks/test_cloud_memorystore.py I can see the code commented out. Is it expected?
   - Can you add this integration to refernece? https://airflow.readthedocs.io/en/latest/operators-and-hooks-ref.html#google
   
   Kind regards 🐈


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