[ 
https://issues.apache.org/jira/browse/HIVE-4816?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sushanth Sowmyan updated HIVE-4816:
-----------------------------------

    Attachment: 4816.patch

Attaching the original exploratory patch I had up on HIVE-4551 that introduces 
notion of a DataTransformer and a TransformedHCatRecord.

This patch still has one important bug that needs to be ironed out - I attach 
it mostly to show what I intended by my previous comment.

The bug is that it still doesn't do conversion of shorts/bytes inside a nested 
structure, and so even if all the tests currently pass with this patch, this 
patch is broken(an indication that we need more tests ). Essentially, a lot 
more of PigHCatUtil.getJavaObj needs to move into the DataTransformer, and out 
of PigHCatUtil.
                
> Followup from HIVE-4551 : Reworking HCat Record Transformation
> --------------------------------------------------------------
>
>                 Key: HIVE-4816
>                 URL: https://issues.apache.org/jira/browse/HIVE-4816
>             Project: Hive
>          Issue Type: Improvement
>            Reporter: Sushanth Sowmyan
>            Assignee: Sushanth Sowmyan
>         Attachments: 4816.patch
>
>
> This is a post-bug followup from HIVE-4551, to refactor some HCatalog code to 
> be cleaner/neater. Reposting contents from my previous jira, this jira needs 
> to track the following:
> a) HCatRecord and HCatSchema reflect underlying raw data and do no promotions.
>  b) Introduce a ConversionImpl, which defines various datatype conversion 
> functions, which all default to returning the input, and having a config that 
> allows a user which conversions are implemented.
>  c) Introduce a PromotedHCatRecord & PromotedHCatSchema that wrap 
> HCatRecord/HCatSchema and use a ConversionImpl.
>  d) Implement a PigLoaderConversionImpl/PigStorerConversionImpl in 
> hcat-pig-adapter, which implements the following: Short->Int promotion, 
> Short->Int promotion, Boolean->Int promotion
>  e) Have HCatLoader/HCatStorer use the promoted versions of 
> HCatRecord/HCatSchema which use the PigConversionImpl.
>  f) Remove the current HCatContext promotion parameters and make them be 
> HCatLoader/HCatStorer parameters.

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