aggregateknowledge / java-hll

Java library for the HyperLogLog algorithm
http://research.neustar.biz/2013/12/24/open-source-release-java-hll/
Apache License 2.0
312 stars 71 forks source link

Large range HLL resulting in 0 cardinality #27

Open JonathanAquino opened 5 years ago

JonathanAquino commented 5 years ago

There are two unit tests SparseHLLTest#largeRangeSmokeTest() and FullHLLTest#largeRangeSmokeTest() that are supposed to test cardinalities of very large HLLs. However, if you insert

System.out.println(cardinality + ", " + expected);

at the bottom of them, you see that both the expected and actual cardinalities are 0. They should instead be a very large number.