[jira] [Commented] (PIG-3067) HBaseStorage should be split up to become more manageable

["Dmitriy V. Ryaboy (JIRA)" <ji...@apache.org>]

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

Dmitriy V. Ryaboy commented on PIG-3067:
----------------------------------------

Just chiming in to say thanks, I like where this is going as well.
                
> HBaseStorage should be split up to become more manageable
> ---------------------------------------------------------
>
>                 Key: PIG-3067
>                 URL: https://issues.apache.org/jira/browse/PIG-3067
>             Project: Pig
>          Issue Type: Improvement
>            Reporter: Christoph Bauer
>            Assignee: Christoph Bauer
>         Attachments: hbasestorage-split.patch
>
>
> HBaseStorage has become quite big (>1100 lines).
> I propose to split it up into more managable parts. I believe it will become a lot easier to maintain.
> I split it up like this:
> HBaseStorage
> * settings:LoadStoreFuncSettings
> ** options
> ** caster
> ** udfProperties
> ** contextSignature
> ** columns:ColumnInfo - moved to its own class-file
> * loadFuncDelegate:HBaseLoadFunc - LoadFunc implementation
> ** settings:LoadStoreFuncSettings (s.a.)
> ** scanner:HBaseLoadFuncScanner - everything scan-specific
> ** tupleIterator:HBaseTupleIterator - interface for _public Tuple getNext()_
> * storeFuncDelegate:HBaseStorFunc - StorFunc implementation
> ** settings:LoadStoreFuncSettings (s.a.)

--
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