Skip to content

[AIRFLOW-6978] Add PubSubPullOperator#7766

Merged
turbaszek merged 27 commits into
apache:masterfrom
PolideaInternal:AIRFLOW-6978-pubsub-pull-operator
Mar 20, 2020
Merged

[AIRFLOW-6978] Add PubSubPullOperator#7766
turbaszek merged 27 commits into
apache:masterfrom
PolideaInternal:AIRFLOW-6978-pubsub-pull-operator

Conversation

@turbaszek

@turbaszek turbaszek commented Mar 19, 2020

Copy link
Copy Markdown
Member

Issue link: AIRFLOW-6978

Make sure to mark the boxes below before creating PR: [x]

  • Description above provides context of the change
  • Commit message/PR title starts with [AIRFLOW-NNNN]. AIRFLOW-NNNN = JIRA ID*
  • Unit tests coverage for changes (not needed for documentation changes)
  • Commits follow "How to write a good git commit message"
  • Relevant documentation is updated including usage instructions.
  • I will engage committers as explained in Contribution Workflow Example.

* For document-only changes commit message can start with [AIRFLOW-XXXX].


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

ANiteckiP and others added 27 commits March 16, 2020 11:56
Co-Authored-By: Kamil Breguła <mik-laj@users.noreply.github.com>
Co-Authored-By: Kamil Breguła <mik-laj@users.noreply.github.com>
Co-Authored-By: Kamil Breguła <mik-laj@users.noreply.github.com>
Co-Authored-By: Kamil Breguła <mik-laj@users.noreply.github.com>
Co-Authored-By: Tomek Urbaszek <turbaszek@gmail.com>
Co-authored-by: nuclearpinguin <turbaszek@apache.org>
@turbaszek turbaszek requested a review from mik-laj March 19, 2020 10:51
@boring-cyborg boring-cyborg Bot added area:docs provider:google Google (including GCP) related issues labels Mar 19, 2020
@codecov-io

Copy link
Copy Markdown

Codecov Report

Merging #7766 into master will decrease coverage by 54.60%.
The diff coverage is 23.80%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master    #7766       +/-   ##
===========================================
- Coverage   86.94%   32.33%   -54.61%     
===========================================
  Files         920      919        -1     
  Lines       44456    44370       -86     
===========================================
- Hits        38651    14346    -24305     
- Misses       5805    30024    +24219     
Impacted Files Coverage Δ
...viders/google/cloud/example_dags/example_pubsub.py 0.00% <0.00%> (-100.00%) ⬇️
airflow/providers/google/cloud/hooks/pubsub.py 22.22% <14.28%> (-72.69%) ⬇️
airflow/providers/google/cloud/sensors/pubsub.py 38.09% <18.75%> (-56.20%) ⬇️
airflow/providers/google/cloud/operators/pubsub.py 29.41% <34.61%> (-61.29%) ⬇️
airflow/sensors/base_sensor_operator.py 43.05% <100.00%> (-55.56%) ⬇️
airflow/hooks/S3_hook.py 0.00% <0.00%> (-100.00%) ⬇️
airflow/hooks/pig_hook.py 0.00% <0.00%> (-100.00%) ⬇️
airflow/hooks/hdfs_hook.py 0.00% <0.00%> (-100.00%) ⬇️
airflow/hooks/http_hook.py 0.00% <0.00%> (-100.00%) ⬇️
airflow/hooks/jdbc_hook.py 0.00% <0.00%> (-100.00%) ⬇️
... and 793 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 97dbc1b...74bb327. Read the comment docs.

@turbaszek turbaszek requested a review from potiuk March 19, 2020 20:42
@turbaszek turbaszek self-assigned this Mar 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

provider:google Google (including GCP) related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants