[ 
https://issues.apache.org/jira/browse/PIG-1409?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12888141#action_12888141
 ] 

Olga Natkovich commented on PIG-1409:
-------------------------------------

SUM.java, MIN.java, MAX.java, COUNT.java, AVG.java, COUNT_STAR.java: 
-----------
 "+ * {...@link org.apache.pig.Algebraic}, so if possible the execution will be 
split into a local and global application." - I think this is a little unclear. 
Perhaps we can just say that it allows the function to be computed in a 
distributed fashion.
 "While this will never be the preferred method of usage it is available in 
case other UDFs can accumulate but not use the combiner. ". I think we might 
want to clarify this by saying that this is not used with combiner or something 
like that.

DIFF.jabva
----------------
"DIFF compares takes two bags as arguments and conpares them" - Need to remove 
first compare and there is a typo in the second

The rest looks good.

+1 once the comments above are addressed.


> Fix up javadocs for org.apache.pig.builtin
> ------------------------------------------
>
>                 Key: PIG-1409
>                 URL: https://issues.apache.org/jira/browse/PIG-1409
>             Project: Pig
>          Issue Type: Sub-task
>          Components: documentation
>            Reporter: Alan Gates
>            Assignee: Alan Gates
>            Priority: Minor
>             Fix For: 0.8.0
>
>         Attachments: PIG-1409.patch
>
>
> There are no external interfaces in this package to mark.  However some of 
> the javadocs can use improvement.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to