This repository has been archived by the owner on Jul 7, 2020. It is now read-only.

Question: What's the purpose of the `size` field in CountMinSketch? #140

Open

alexandrnikitin opened this issue Aug 22, 2017 · 0 comments

alexandrnikitin commented Aug 22, 2017 •

edited

Loading

Why can't I do something like this:

        CountMinSketch sketch = new CountMinSketch(0.00001, 0.99999, 1);
        sketch.add(1, Long.MAX_VALUE);
        sketch.add(2, 22);

It fails because of the overflow check.

The text was updated successfully, but these errors were encountered:

alexandrnikitin mentioned this issue

CountMinSketch overflow check causes a lot of allocations #141

Open

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.