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

Ashutosh Chauhan commented on HIVE-11985:
-----------------------------------------

[~sershe] Can you confirm my understanding of the patch?
* If serde doesnt belong in config {{hive.serdes.using.metastore.for.schema}} 
schema is always stored in metastore. Patch introduces a check which will 
disallow type > 2K for this case.
* If serde doesn't belong in that config, then we enquire serde whether he 
wants to store its schema or not in metastore. If it say yes, then we store it 
with the above check.  Else, we store {{erived from deserializer}} as type for 
it.

I think this strategy does make sense. We can additionally get rid of  config 
{{hive.serdes.using.metastore.for.schema}} now that a method of same effect has 
been added.

> don't store type names in metastore when metastore type names are not used
> --------------------------------------------------------------------------
>
>                 Key: HIVE-11985
>                 URL: https://issues.apache.org/jira/browse/HIVE-11985
>             Project: Hive
>          Issue Type: Bug
>            Reporter: Sergey Shelukhin
>            Assignee: Sergey Shelukhin
>         Attachments: HIVE-11985.01.patch, HIVE-11985.02.patch, 
> HIVE-11985.03.patch, HIVE-11985.05.patch, HIVE-11985.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to