[jira] [Commented] (PIG-4852) Add accumulator implementation for MaxTupleBy1stField

2016-07-22 Thread Adam Szita (JIRA)
[ https://issues.apache.org/jira/browse/PIG-4852?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15389121#comment-15389121 ] Adam Szita commented on PIG-4852: - Thanks for spotting this Daniel - patch revised > Add ac

[jira] [Commented] (PIG-4852) Add accumulator implementation for MaxTupleBy1stField

2016-07-21 Thread Daniel Dai (JIRA)
[ https://issues.apache.org/jira/browse/PIG-4852?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15388945#comment-15388945 ] Daniel Dai commented on PIG-4852: - You also need to change javac.version to 1.7 in contrib/

[jira] [Commented] (PIG-4852) Add accumulator implementation for MaxTupleBy1stField

2016-07-20 Thread Adam Szita (JIRA)
[ https://issues.apache.org/jira/browse/PIG-4852?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15385906#comment-15385906 ] Adam Szita commented on PIG-4852: - This makes sense Eyal, I'll look into it > Add accumulat

[jira] [Commented] (PIG-4852) Add accumulator implementation for MaxTupleBy1stField

2016-07-20 Thread Eyal Allweil (JIRA)
[ 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 sinc

[jira] [Commented] (PIG-4852) Add accumulator implementation for MaxTupleBy1stField

2016-07-20 Thread Adam Szita (JIRA)
[ https://issues.apache.org/jira/browse/PIG-4852?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15385473#comment-15385473 ] Adam Szita commented on PIG-4852: - Hi, Isn't this already covered by ExternalTupleByNthFiel