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

Josh Wills commented on CRUNCH-323:
-----------------------------------

+1, nice explanation of the details that motivate the functionality.

Couple of nits-- there's an "it's" that needs to be "its", and in for the 
formatting of the pre-code block to work correctly, you need to swap in < 
and &gt; for the PCollection<T> and Emitter<T>.

> Add a section on getDetachedValue to the user guide
> ---------------------------------------------------
>
>                 Key: CRUNCH-323
>                 URL: https://issues.apache.org/jira/browse/CRUNCH-323
>             Project: Crunch
>          Issue Type: Improvement
>          Components: Site
>            Reporter: Gabriel Reid
>            Assignee: Gabriel Reid
>            Priority: Minor
>         Attachments: getDetachedValue.patch
>
>
> The use of the PType#getDetachedValue is a useful bit of information, and a 
> common gotcha when working with Crunch and MapReduce. A section explaining 
> how to deal with retaining references within a DoFn should be added to the 
> user guide.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Reply via email to