[ 
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

Reply via email to