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

Change type of TDigest.Group.id from int to long

Open sgrj opened this issue 5 years ago • 1 comments

The assumption is that the id is unique. If the id value overflows, it can lead to NullPointerExceptions. Changing the type to long doesn't make overflows impossible, but highly unlikely.

This fixes issues like #100

sgrj avatar Oct 29 '19 14:10 sgrj

Thanks, switching to MergingDigest indeed shows some nice performance improvements.

sgrj avatar Oct 30 '19 08:10 sgrj