hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Carl Steinbach (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HIVE-3410) All operators's conf should inherit from a common class
Date Wed, 29 Aug 2012 07:47:07 GMT

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

Carl Steinbach commented on HIVE-3410:
--------------------------------------

@Namit: It looks like the ObjectPair class from Navis's recent patch was accidentally included
in the most recent update. Can you please remove this? Otherwise, everything looks good to
me. Can you please handle testing and committing this? Thanks.

+1

                
> All operators's conf should inherit from a common class
> -------------------------------------------------------
>
>                 Key: HIVE-3410
>                 URL: https://issues.apache.org/jira/browse/HIVE-3410
>             Project: Hive
>          Issue Type: Improvement
>            Reporter: Namit Jain
>            Assignee: Namit Jain
>         Attachments: hive.3410.1.patch, hive.3410.2.patch
>
>
> I faced this problem while working on HIVE-3086.
> Wanted to add a common interface to all the operator's confs.
> Currently, the operator signature is Operator<T extends Serializable>,
> which makes it difficult to add more methods to T.
> Created a common interface SerializableCloneable, and now 
> Operator<T extends SerializableCloneable>.
> If we want to add more methods to SerializableCloneable in the future,
> it wont require changing so many files, so submitting it as a separate patch.

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