pig-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "pablo martinez (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (PIG-3057) make readField protected to be able to override it if we extend PigStorage
Date Mon, 07 Jan 2013 16:50:17 GMT

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

pablo martinez commented on PIG-3057:
-------------------------------------

thanks for the explanation. So I'm good with your proposal as it is
thank you very much




                
> make readField protected to be able to override it if we extend PigStorage
> --------------------------------------------------------------------------
>
>                 Key: PIG-3057
>                 URL: https://issues.apache.org/jira/browse/PIG-3057
>             Project: Pig
>          Issue Type: Improvement
>          Components: build, internal-udfs
>    Affects Versions: 0.8.0, 0.8.1, 0.9.0, 0.9.1, 0.9.2, 0.10.0
>            Reporter: pablo martinez
>            Priority: Trivial
>              Labels: patch
>         Attachments: PIG-3057_1.patch, PigStorage_readField.patch
>
>   Original Estimate: 2h
>  Remaining Estimate: 2h
>
> for the cases when we need to extend PigStorage just to override readField. Currently,
we need to copy/paste several private fields and all getNext
> I've changed readField from private to protected and added a new method: protected void
addToCurrentTuple(DataByteArray data) 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Mime
View raw message