[
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
