Issue #326 - Some HyperLogLog tests are not testing what we think.

[enricoap]

Issue #326

Proposed Changes for the jRhoW test

• the size of the Array[Byte] in input in input of the jRhoW function is now dependent on the number of bits.

[CLAassistant]

All committers have signed the CLA.

[johnynek]

Thanks for the PR.

Can you run scalafmt at the sbt prompt?

I'm happy to merge this. I guess previously we just never hit a case that had to go very far into the array.

[enricoap]

Sorry, I just saw this comment now. Will do!

[codecov-io]

Codecov Report

Merging #923 (9c141e4) into develop (ff8b0e6) will decrease coverage by 0.08%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #923      +/-   ##
===========================================
- Coverage    89.67%   89.59%   -0.09%     
===========================================
  Files          119      119              
  Lines         9869     9869              
  Branches       494      494              
===========================================
- Hits          8850     8842       -8     
- Misses        1019     1027       +8     

Impacted Files	Coverage Δ
...main/scala/com/twitter/algebird/monad/Reader.scala	52.94% <0.00%> (-5.89%)	⬇️
...src/main/scala/com/twitter/algebird/Interval.scala	75.65% <0.00%> (-5.22%)	⬇️
...a/com/twitter/algebird/immutable/BloomFilter.scala	95.08% <0.00%> (-0.82%)	⬇️
.../scala/com/twitter/algebird/immutable/BitSet.scala	96.57% <0.00%> (-0.22%)	⬇️
.../main/scala/com/twitter/algebird/BloomFilter.scala	94.24% <0.00%> (+0.44%)	⬆️

---

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 ff8b0e6...9c141e4. Read the comment docs.

[johnynek]

Thank you!