tanjinP commented on pull request #10121: URL: https://github.com/apache/airflow/pull/10121#issuecomment-683768310
> 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 🐈 Hey @mschickensoup - I've addressed some of the feedback that you pointed out. Thanks. As for the `TODO` on the tests - I was waiting for the initial feedback before I implement some more operators and hooks. Those are yet to be done and I'll get on it later tonight ([specifically for the instance resource](https://cloud.google.com/memorystore/docs/memcached/reference/rest/v1beta2/projects.locations.instances)). Will @ you once that is complete. Thanks! ---------------------------------------------------------------- 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]
