datafu-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Eyal Allweil (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (DATAFU-114) Make FirstTupleFromBag implement Accumulator
Date Sat, 06 Feb 2016 11:07:39 GMT

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

Eyal Allweil commented on DATAFU-114:
-------------------------------------

The test looks fine, and so does your patch for DATAFU-95 - I was able to build and test (after
adding the test to BagTests.java). What I still can't do is get an Eclipse project working
- the gradlew completes, but the project which results doesn't have source folders or dependencies.

In the past I had trouble generating patches from Git which RB accepted, but maybe that's
been taken care of.

Thanks!

> Make FirstTupleFromBag implement Accumulator
> --------------------------------------------
>
>                 Key: DATAFU-114
>                 URL: https://issues.apache.org/jira/browse/DATAFU-114
>             Project: DataFu
>          Issue Type: Improvement
>    Affects Versions: 1.3.0
>         Environment: All
>            Reporter: Eyal Allweil
>            Assignee: Eyal Allweil
>            Priority: Minor
>              Labels: easyfix, newbie, performance
>             Fix For: 1.3.1
>
>         Attachments: FirstTupleFromBag.java
>
>
> FirstTupleFromBag only needs the first tuple from the bag, but because it doesn't implement
Accumulator the entire bag needs to be passed to it in-memory. The fix is very minor and will
make the UDF support large bags.



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

Mime
View raw message