-
Notifications
You must be signed in to change notification settings - Fork 130
Python: use google.api.core.Operation for long running operations. #1509
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1509 +/- ##
============================================
+ Coverage 86.92% 86.92% +<.01%
- Complexity 4112 4116 +4
============================================
Files 404 404
Lines 16575 16585 +10
Branches 1762 1764 +2
============================================
+ Hits 14407 14417 +10
Misses 1683 1683
Partials 485 485
Continue to review full report at Codecov.
|
@@ -11,6 +11,7 @@ import sys | |||
|
|||
install_requires = [ | |||
'google-auth>=1.0.2, <2.0dev', | |||
'google-cloud-core>=0.27.0, <1.0dev', |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was not expecting you to do this -- thank you! One minor change. It is LGTM'd after that.
@@ -131,6 +131,9 @@ public boolean hasMissingDefaultOptions() { | |||
@Nullable | |||
public abstract String gapicPackageName(); | |||
|
|||
@Nullable | |||
public abstract List<String> operationTypes(); |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This is now obsolete. |
Updates #1320
Here is the Operation class for reference:
https://github.com/GoogleCloudPlatform/google-cloud-python/blob/master/core/google/api/core/operation.py