hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Tongjie Chen (JIRA)" <>
Subject [jira] [Commented] (HIVE-6785) query fails when partitioned table's table level serde is ParquetHiveSerDe and partition level serde is of different SerDe
Date Sat, 05 Apr 2014 21:21:18 GMT


Tongjie Chen commented on HIVE-6785:

In my previous comment, I mean that this bug is not reproducible in hive trunk due to patches
from HIVE-5202 and HIVE-5394.

But the patch introduced in this Jira is still an enhancement.

Btw, how can I edit my own previous comment once submitted?

> query fails when partitioned table's table level serde is ParquetHiveSerDe and partition
level serde is of different SerDe
> --------------------------------------------------------------------------------------------------------------------------
>                 Key: HIVE-6785
>                 URL:
>             Project: Hive
>          Issue Type: Bug
>          Components: File Formats, Serializers/Deserializers
>    Affects Versions: 0.13.0
>            Reporter: Tongjie Chen
>         Attachments: HIVE-6785.1.patch.txt
> When a hive table's SerDe is ParquetHiveSerDe, while some partitions are of other SerDe,
AND if this table has string column[s], hive generates confusing error message:
> "Failed with exception parquet.hive.serde.primitive.ParquetStringInspector
cannot be cast to org.apache.hadoop.hive.serde2.objectinspector.primitive.SettableTimestampObjectInspector"
> This is confusing because timestamp is mentioned even if it is not been used by the table.
The reason is when there is SerDe difference between table and partition, hive tries to convert
objectinspector of two SerDes. ParquetHiveSerDe's object inspector for string type is ParquetStringInspector
(newly introduced), neither a subclass of WritableStringObjectInspector nor JavaStringObjectInspector,
which ObjectInspectorConverters expect for string category objector inspector. There is no
break statement in STRING case statement, hence the following TIMESTAMP case statement is
executed, generating confusing error message.
> see also in the following parquet issue:
> To fix that it is relatively easy, just make ParquetStringInspector subclass of JavaStringObjectInspector
instead of AbstractPrimitiveJavaObjectInspector. But because constructor of class JavaStringObjectInspector
is package scope instead of public or protected, we would need to move ParquetStringInspector
to the same package with JavaStringObjectInspector.
> Also ArrayWritableObjectInspector's setStructFieldData needs to also accept List data,
since the corresponding setStructFieldData and create methods return a list. This is also
needed when table SerDe is ParquetHiveSerDe, and partition SerDe is something else.

This message was sent by Atlassian JIRA

View raw message