[
https://issues.apache.org/jira/browse/PIG-4852?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15385567#comment-15385567
]
Eyal Allweil commented on PIG-4852:
-----------------------------------
Adam - What you're saying is of course true. But since "MaxTupleBy1stField"
exists for backwards compatibility/convenience, and some people
(must)(probably?) use it, I think it's worth it to add the Accumulator
implementation and speed things up for such cases.
Another option might be to make MaxTupleBy1stField use ExtremalTupleByNthField
instead of having its own implementation.
> Add accumulator implementation for MaxTupleBy1stField
> -----------------------------------------------------
>
> Key: PIG-4852
> URL: https://issues.apache.org/jira/browse/PIG-4852
> Project: Pig
> Issue Type: Improvement
> Reporter: Eyal Allweil
> Priority: Minor
> Labels: newbie, performance
>
> MaxTupleBy1stField has a regular and algebraic implementation; an accumulator
> implementation should be relatively easy to do.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
