Robin Sommer commented on BIT-1072:
I ended up refactoring and reformatting this quite a bit, it's in
topic/robin/hyperlolog-merge. However, I broke something, the tests aren't working.
Need to debug that later. In the meantime, some requests/questions:
- Please look over my changes and see if they make sense. (You don't need to track
down the bug; I take the blame for that :).
- Can you please rework the Doxygen comments in HyperLogLog.h so that the descriptions for
the public methods are understandable on their own. Right now I can't really follow
them as often they talk about internal parameters/functionality. What you could do is
provide a short overview of the data structure parameters in the class' doc string,
and then refer to that in the methods. Also, please use the @param and @return syntax.
(Start from my branch with this: I already reformatted and reordered things there quite a
- I don't understand what can be parameterized by the user and what not (and why
not). One can give an error margin to the actor, but the confidence is a compile time
constant. Also, where are the magic alpha_m values in *.cc coming from? Are these indeed
always static values that don't depend on any parameters?
Project: Bro Issue Tracker
Issue Type: New Feature
Affects Versions: git/master
Reporter: Bernhard Amann
Fix For: 2.2
The branch adds support for the hyperloglog data structure.
In the branch, core/leaks/basic-cluster.bro currently faisl. However, this seems to be
unrelated to hll and just to be triggered by the addition of it to the sumstats tests. It
looks like some kind of scriptland issue. pprof output is attached. (master, workers
don't leak memory)
This message was sent by Atlassian JIRA