datafu-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ido Hadanny (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (DATAFU-91) pig version of HyperLogLog estimator should be Algebraic and use combiners
Date Sat, 22 Aug 2015 09:11:45 GMT

    [ https://issues.apache.org/jira/browse/DATAFU-91?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14707953#comment-14707953
] 

Ido Hadanny commented on DATAFU-91:
-----------------------------------

Hey [~matterhayes], sorry but I just can't seem to complete this patch :(
Tried building on my windows machine and ran into a gradle/windows problem. https://discuss.gradle.org/t/error-with-a-copy-task-on-windows/1803


Please re-assign this issue to yourself and complete it - its just changing the `offer` to
`offerHashed` and making sure everything works. 
Looking forward to see this in the next datafu version!

> pig version of HyperLogLog estimator should be Algebraic and use combiners
> --------------------------------------------------------------------------
>
>                 Key: DATAFU-91
>                 URL: https://issues.apache.org/jira/browse/DATAFU-91
>             Project: DataFu
>          Issue Type: Bug
>    Affects Versions: 1.3.0
>            Reporter: Ido Hadanny
>            Assignee: Ido Hadanny
>            Priority: Minor
>             Fix For: 1.3.0
>
>         Attachments: hyper-log-log-algebraic-3.diff, hyper-log-log-algebraic.diff, hyper-log-log-algebraic.diff
>
>
> Matt: I don't remember if there was a particular reason I didn't implement this as AlgebraicEvalFunc.
It seems like it could be. I believe the Java MapReduce version leverages the combiner. If
you want to try making this Algebraic we would be happy to accept a patch :)



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message