addthis / stream-lib

Stream summarizer and cardinality estimator.
Apache License 2.0
2.26k stars 556 forks source link

Generic type for Count-Min Sketch #139

Open arnaudfnr opened 7 years ago

arnaudfnr commented 7 years ago

Hi guys,

First of all thank you the great work you put in this library !

I am using Count-Min sketch implementation and I wonder why the only supported types are "long" and "String". Indeed, there are some types of object such as Doubles or Integers that are easily and for sure convertible to String. For Objects in general, the method toString() could be called, like in the Murmurhash used in HyperLogLog. Is there a specific reason that you decided not to use it for CountMinSketch ?

Best regards,

Arnaud