[jira] [Commented] (HIVE-4844) Add char/varchar data types

["Jason Dere (JIRA)" <ji...@apache.org>]

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

Jason Dere commented on HIVE-4844:
----------------------------------

Added a doc to the wiki at https://cwiki.apache.org/confluence/display/Hive/Type+Qualifiers+in+Hive. 

Ed, for the metastore changes, would additional columns in the COLUMNS_V2 metastore table be fine, if they are set to NULLs if there are no type qualifiers?

Xuefu, yeah for char/varchar I was thinking they could probably share some inheritance, especially since they need to do some length checking, though I hadn't really thought about if they should share with existing String/Text. Well, they can't inherit String as I mentioned before.  For decimal/decimal(p,s), I think the  existing HiveDecimal might be fine, since I think the actual enforcement of precision/scale might happen at the ObjectInspectors/Converters/cast/deserialization. 
                
> Add char/varchar data types
> ---------------------------
>
>                 Key: HIVE-4844
>                 URL: https://issues.apache.org/jira/browse/HIVE-4844
>             Project: Hive
>          Issue Type: New Feature
>          Components: Types
>            Reporter: Jason Dere
>            Assignee: Jason Dere
>         Attachments: HIVE-4844.1.patch.hack
>
>
> Add new char/varchar data types which have support for more SQL-compliant behavior, such as SQL string comparison semantics, max length, etc.

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