[ 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)