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

Ashutosh Chauhan commented on HIVE-6756:
----------------------------------------

Patch looks good, accept for textfile & seqfile 
+      serde = conf.getVar(HiveConf.ConfVars.HIVESCRIPTSERDE);

scriptserde config is used for other purposes, I think its better just to do 
serde = LazySimpleSerDe.getClass().getName() 
since thats equivalent behavior with create table stored as textfile / 
sequencefile

Looks good otherwise 

> alter table set fileformat should set serde too
> -----------------------------------------------
>
>                 Key: HIVE-6756
>                 URL: https://issues.apache.org/jira/browse/HIVE-6756
>             Project: Hive
>          Issue Type: Bug
>    Affects Versions: 0.13.0
>            Reporter: Owen O'Malley
>            Assignee: Chinna Rao Lalam
>         Attachments: HIVE-6756.1.patch, HIVE-6756.2.patch, HIVE-6756.patch
>
>
> Currently doing alter table set fileformat doesn't change the serde. This is 
> unexpected by customers because the serdes are largely file format specific.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to