-
Notifications
You must be signed in to change notification settings - Fork 13.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[AIRFLOW-6111] [AIP-21] Rename GCP spanner operator and hook #7004
[AIRFLOW-6111] [AIP-21] Rename GCP spanner operator and hook #7004
Conversation
Codecov Report
@@ Coverage Diff @@
## master #7004 +/- ##
==========================================
- Coverage 85.15% 84.88% -0.28%
==========================================
Files 680 680
Lines 38824 38848 +24
==========================================
- Hits 33061 32975 -86
- Misses 5763 5873 +110
Continue to review full report at Codecov.
|
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.
LGTM just unrelated changes in UPDATING.md
UPDATING.md
Outdated
|airflow.contrib.operators.gcp_natural_language_operator.CloudLanguageAnalyzeEntitiesOperator |airflow.providers.google.cloud.operators.natural_language.CloudLanguageAnalyzeEntitiesOperator | | ||
|airflow.contrib.operators.gcp_natural_language_operator.CloudLanguageAnalyzeEntitySentimentOperator |airflow.providers.google.cloud.operators.natural_language.CloudLanguageAnalyzeEntitySentimentOperator | | ||
|airflow.contrib.operators.gcp_natural_language_operator.CloudLanguageAnalyzeSentimentOperator |airflow.providers.google.cloud.operators.natural_language.CloudLanguageAnalyzeSentimentOperator | | ||
|airflow.contrib.operators.gcp_natural_language_operator.CloudLanguageClassifyTextOperator |airflow.providers.google.cloud.operators.natural_language.CloudLanguageClassifyTextOperator | |
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 think this are unrelated changes?
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.
What about new paths?
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.
Yeah, but this changes CloudLanguage
and the PR says Spanner
💃
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.
Ops... Fix is on the way.
278644e
to
06f5993
Compare
06f5993
to
fccc34d
Compare
fccc34d
to
033efd0
Compare
PR contains changes regarding AIP-21 (renaming GCP operators and hooks):
renamed GCP modules
adde deprecation warnings to the contrib modules
fixed tests
updated UPDATING.md
[AIRFLOW-XXXX]
for document-only changesIn case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.
Read the Pull Request Guidelines for more information.