Skip to content

[AIRFLOW-7088] Avoid reading store_dag_code option in loop#7764

Merged
mik-laj merged 1 commit intoapache:masterfrom
PolideaInternal:AIRFLOW-7088
Mar 19, 2020
Merged

[AIRFLOW-7088] Avoid reading store_dag_code option in loop#7764
mik-laj merged 1 commit intoapache:masterfrom
PolideaInternal:AIRFLOW-7088

Conversation

@mik-laj
Copy link
Member

@mik-laj mik-laj commented Mar 19, 2020


Issue link: AIRFLOW-7088

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.

@boring-cyborg boring-cyborg bot added the area:Scheduler including HA (high availability) scheduler label Mar 19, 2020
@codecov-io
Copy link

codecov-io commented Mar 19, 2020

Codecov Report

Merging #7764 into master will decrease coverage by 0.25%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #7764      +/-   ##
==========================================
- Coverage   86.93%   86.67%   -0.26%     
==========================================
  Files         920      920              
  Lines       44447    44448       +1     
==========================================
- Hits        38640    38526     -114     
- Misses       5807     5922     +115     
Impacted Files Coverage Δ
airflow/utils/dag_processing.py 88.04% <100.00%> (+0.02%) ⬆️
airflow/kubernetes/volume_mount.py 44.44% <0.00%> (-55.56%) ⬇️
airflow/kubernetes/volume.py 52.94% <0.00%> (-47.06%) ⬇️
airflow/kubernetes/pod_launcher.py 47.18% <0.00%> (-45.08%) ⬇️
...viders/cncf/kubernetes/operators/kubernetes_pod.py 69.69% <0.00%> (-25.26%) ⬇️
airflow/kubernetes/refresh_config.py 50.98% <0.00%> (-23.53%) ⬇️
airflow/jobs/backfill_job.py 92.11% <0.00%> (+1.12%) ⬆️

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 cc5376d...903877e. Read the comment docs.

@mik-laj mik-laj merged commit d9eb0be into apache:master Mar 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:Scheduler including HA (high availability) scheduler

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments