hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ashutosh Chauhan (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HIVE-4007) Create abstract classes for serializer and deserializer
Date Tue, 19 Feb 2013 06:47:13 GMT

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

Ashutosh Chauhan commented on HIVE-4007:
----------------------------------------

Whats exactly the incompatibility here? Since AbstractSerde implements Serde. Any existing
serde should still continue to work without needing to recompile because in codebase we are
still refering by Serde, not by AbstractSerde. Existing serde can move to AbstractSerde as
they see fit. New serdes we can recommend to extend AbstractSerde. When we do add new apis
to AbstractSerde, we need to make sure we call them on serde only if they implement. This
is not clean, but can be made to work.

That aside, Namit can you point out the motivation for this. Which new apis you want to add
in AbstractSerde? At the moment, there is none.
                
> Create abstract classes for serializer and deserializer
> -------------------------------------------------------
>
>                 Key: HIVE-4007
>                 URL: https://issues.apache.org/jira/browse/HIVE-4007
>             Project: Hive
>          Issue Type: Improvement
>          Components: Serializers/Deserializers
>            Reporter: Namit Jain
>            Assignee: Namit Jain
>         Attachments: hive.4007.1.patch, hive.4007.2.patch
>
>
> Currently, it is very difficult to change the Serializer/Deserializer
> interface, since all the SerDes directly implement the interface.
> Instead, we should have abstract classes for implementing these interfaces.
> In case of a interface change, only the abstract class and the relevant 
> serde needs to change.

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