Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

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

Closed
wants to merge 6 commits into from

Conversation

ziafazal
Copy link
Contributor

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 ziafazal changed the title We are using Hive 2.1.1 and NULLIF is support in 2.3.0 onwards We are using Hive 2.1.1 and NULLIF is supported in 2.3.0 onwards Sep 13, 2019
@codecov
Copy link

codecov bot commented Sep 13, 2019

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% <ø> (ø) ⬆️
...alytics/tasks/enterprise/enterprise_enrollments.py 83.87% <ø> (ø) ⬆️
...alytics/tasks/tests/acceptance/test_enrollments.py 28.42% <0%> (-0.31%) ⬇️

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.

Fixed broken test

Fixed another broken test
@ziafazal
Copy link
Contributor Author

I don't think this is needed now.

@ziafazal ziafazal closed this Jan 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant