openedx-unsupported / edx-analytics-pipeline

GNU Affero General Public License v3.0
91 stars 116 forks source link

Clean up error handling in mysql_load temp tables #752

Closed bmedx closed 5 years ago

bmedx commented 5 years ago

Analytics Pipeline Pull Request

Make sure that the following steps are done before merging:

codecov[bot] commented 5 years ago

Codecov Report

Merging #752 into master will decrease coverage by <.01%. The diff coverage is 70%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #752      +/-   ##
==========================================
- Coverage   74.57%   74.56%   -0.01%     
==========================================
  Files         207      207              
  Lines       23404    23411       +7     
==========================================
+ Hits        17453    17457       +4     
- Misses       5951     5954       +3
Impacted Files Coverage Δ
edx/analytics/tasks/common/mysql_load.py 70% <70%> (-0.38%) :arrow_down:

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 c28f775...3de9b01. Read the comment docs.