openedx-unsupported / edx-analytics-pipeline

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

fix: Re-add load_internal_reporting_country.py #862

Closed bmedx closed 3 years ago

bmedx commented 3 years ago

This was removed in Vertica deprecation, but is still needed. I'm re-adding it, minus the Vertica bits.

Analytics Pipeline Pull Request

Make sure that the following steps are done before merging:

codecov[bot] commented 3 years ago

Codecov Report

Merging #862 (0782403) into master (c014112) will decrease coverage by 0.00%. The diff coverage is 70.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #862      +/-   ##
==========================================
- Coverage   76.43%   76.42%   -0.01%     
==========================================
  Files         151      152       +1     
  Lines       17170    17190      +20     
==========================================
+ Hits        13124    13138      +14     
- Misses       4046     4052       +6     
Impacted Files Coverage Δ
...tasks/warehouse/load_internal_reporting_country.py 70.00% <70.00%> (ø)

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 c014112...0782403. Read the comment docs.