openedx-unsupported / edx-analytics-pipeline

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

Add descriptive fail log upon init failure. #795

Closed doctoryes closed 3 years ago

doctoryes commented 4 years ago

Analytics Pipeline Pull Request

When the LoadInsightsToVertica command fails upon one of the dependency checks, it's currently difficult to tell which table failed. This log will help the next person debugging this task.

Make sure that the following steps are done before merging:

codecov[bot] commented 4 years ago

Codecov Report

Merging #795 into master will decrease coverage by 0.01%. The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #795      +/-   ##
==========================================
- Coverage   74.53%   74.52%   -0.02%     
==========================================
  Files         211      211              
  Lines       24193    24197       +4     
==========================================
  Hits        18032    18032              
- Misses       6161     6165       +4
Impacted Files Coverage Δ
...alytics/tasks/warehouse/load_warehouse_insights.py 55.76% <0%> (-4.65%) :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 b32712a...2059b14. Read the comment docs.