edx-analytics-pipeline icon indicating copy to clipboard operation
edx-analytics-pipeline copied to clipboard

We are using Hive 2.1.1 and NULLIF is supported in 2.3.0 onwards

Open ziafazal opened this issue 5 years ago • 1 comments

Analytics Pipeline Pull Request

Make sure that the following steps are done before merging:

  • [ ] If you have a migration please contact data engineering team before merging.
  • [ ] Before merging run full acceptance tests suite and provide URL for the acceptance tests run.
  • [ ] A member of data engineering team has approved the pull request.

ziafazal avatar Sep 13 '19 11:09 ziafazal

Codecov Report

Merging #758 into master will decrease coverage by <.01%. The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #758      +/-   ##
==========================================
- Coverage   74.55%   74.55%   -0.01%     
==========================================
  Files         207      207              
  Lines       23420    23421       +1     
==========================================
  Hits        17461    17461              
- Misses       5959     5960       +1
Impacted Files Coverage Δ
...ks/tests/acceptance/test_enterprise_enrollments.py 40% <ø> (ø) :arrow_up:
...alytics/tasks/enterprise/enterprise_enrollments.py 83.87% <ø> (ø) :arrow_up:
...alytics/tasks/tests/acceptance/test_enrollments.py 28.42% <0%> (-0.31%) :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 25e870f...4ecb327. Read the comment docs.

codecov[bot] avatar Sep 13 '19 11:09 codecov[bot]

I don't think this is needed now.

ziafazal avatar Jan 15 '24 06:01 ziafazal