-
Notifications
You must be signed in to change notification settings - Fork 28.1k
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
[SPARK-12818][SQL] Specializes integral and string types for Count-min Sketch #10968
Conversation
@@ -124,6 +124,18 @@ int getVersionNumber() { | |||
*/ | |||
public abstract void add(Object item, long count); | |||
|
|||
public abstract void addLong(long item); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
need to add java doc.
also update the other java doc to say "Increment item's count by one." or "Increment item's count by count"
Test build #50266 has finished for PR 10968 at commit
|
@rxin Updated, thanks for the review! |
03ea283
to
0d3d009
Compare
0d3d009
to
463f68a
Compare
LGTM |
Just reverted all accidental formatting changes made by IntelliJ. |
Test build #2468 has finished for PR 10968 at commit
|
Test build #50283 has finished for PR 10968 at commit
|
LGTM |
I'm going to merge this. Thanks. |
This PR is a follow-up of #10911. It adds specialized update methods for
CountMinSketch
so that we can avoid doing internal/external row format conversion inDataFrame.countMinSketch()
.