stream-lib icon indicating copy to clipboard operation
stream-lib copied to clipboard

Merge two TDigests throws NullPointerException

Open kevindai0126 opened this issue 9 years ago • 1 comments

Sometimes when I merged two tdigests by call the add method of one tdigest, it threw NullPointerException as follows: java.lang.NullPointerException at com.clearspring.analytics.stream.quantile.GroupTree.add(GroupTree.java:85) at com.clearspring.analytics.stream.quantile.GroupTree.add(GroupTree.java:79) at com.clearspring.analytics.stream.quantile.GroupTree.add(GroupTree.java:79) at com.clearspring.analytics.stream.quantile.GroupTree.add(GroupTree.java:79) at com.clearspring.analytics.stream.quantile.GroupTree.add(GroupTree.java:79) at com.clearspring.analytics.stream.quantile.GroupTree.add(GroupTree.java:79) at com.clearspring.analytics.stream.quantile.GroupTree.add(GroupTree.java:81) at com.clearspring.analytics.stream.quantile.GroupTree.add(GroupTree.java:81) at com.clearspring.analytics.stream.quantile.GroupTree.add(GroupTree.java:81) at com.clearspring.analytics.stream.quantile.GroupTree.add(GroupTree.java:79) at com.clearspring.analytics.stream.quantile.GroupTree.add(GroupTree.java:81) at com.clearspring.analytics.stream.quantile.GroupTree.add(GroupTree.java:79) at com.clearspring.analytics.stream.quantile.GroupTree.add(GroupTree.java:81) at com.clearspring.analytics.stream.quantile.GroupTree.add(GroupTree.java:81) at com.clearspring.analytics.stream.quantile.TDigest.add(TDigest.java:146) at com.clearspring.analytics.stream.quantile.TDigest.add(TDigest.java:171)

kevindai0126 avatar Nov 20 '15 09:11 kevindai0126

Streamlib should use the AvlTreeDigest instead.

Faster. Better.

On Fri, Nov 20, 2015 at 5:46 PM, kevindai0126 [email protected] wrote:

Sometimes when I merged two tdigests by call the add method of one tdigest, it threw NullPointerException as follows: java.lang.NullPointerException at com.clearspring.analytics.stream.quantile.GroupTree.add(GroupTree.java:85) at com.clearspring.analytics.stream.quantile.GroupTree.add(GroupTree.java:79) at com.clearspring.analytics.stream.quantile.GroupTree.add(GroupTree.java:79) at com.clearspring.analytics.stream.quantile.GroupTree.add(GroupTree.java:79) at com.clearspring.analytics.stream.quantile.GroupTree.add(GroupTree.java:79) at com.clearspring.analytics.stream.quantile.GroupTree.add(GroupTree.java:79) at com.clearspring.analytics.stream.quantile.GroupTree.add(GroupTree.java:81) at com.clearspring.analytics.stream.quantile.GroupTree.add(GroupTree.java:81) at com.clearspring.analytics.stream.quantile.GroupTree.add(GroupTree.java:81) at com.clearspring.analytics.stream.quantile.GroupTree.add(GroupTree.java:79) at com.clearspring.analytics.stream.quantile.GroupTree.add(GroupTree.java:81) at com.clearspring.analytics.stream.quantile.GroupTree.add(GroupTree.java:79) at com.clearspring.analytics.stream.quantile.GroupTree.add(GroupTree.java:81) at com.clearspring.analytics.stream.quantile.GroupTree.add(GroupTree.java:81) at com.clearspring.analytics.stream.quantile.TDigest.add(TDigest.java:146) at com.clearspring.analytics.stream.quantile.TDigest.add(TDigest.java:171)

— Reply to this email directly or view it on GitHub https://github.com/addthis/stream-lib/issues/100.

tdunning avatar Nov 20 '15 10:11 tdunning