Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add HyperLogLog support (PFADD, PFCOUNT, PFMERGE) #1435

Merged
merged 2 commits into from
Feb 21, 2016

Conversation

yannick
Copy link
Contributor

@yannick yannick commented Feb 21, 2016

No description provided.

/// Adds all the element arguments to the HyperLogLog data structure stored at the variable name specified as first argument.
long pfadd(ARGS...)(string key, ARGS args) { return request!long("PFADD", key, args); }
/// When called with a single key, returns the approximated cardinality computed by the HyperLogLog data structure stored at the specified variable, which is 0 if the variable does not exist.
/// When called with multiple keys, returns the approximated cardinality of the union of the HyperLogLogs passed, by internally merging the HyperLogLogs stored at the provided keys into a temporary HyperLogLog.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This needs to get a short summary paragraph as the first line, so that it appears correctly inside of the API docs.

s-ludwig added a commit that referenced this pull request Feb 21, 2016
add HyperLogLog support (PFADD, PFCOUNT, PFMERGE)
@s-ludwig s-ludwig merged commit 25ee729 into vibe-d:master Feb 21, 2016
@s-ludwig
Copy link
Member

I've made the changes as I originally meant in cacc505. At some point I'll have to go through the whole Redis module and fix the formatting + add references to the redis.io docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants