edx-analytics-pipeline
edx-analytics-pipeline copied to clipboard
We are using Hive 2.1.1 and NULLIF is supported in 2.3.0 onwards
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.
Codecov Report
Merging #758 into master will decrease coverage by
<.01%
. The diff coverage is0%
.
@@ 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.
I don't think this is needed now.