You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hive.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2018/10/29 22:49:00 UTC

[jira] [Commented] (HIVE-20827) Inconsistent results for empty arrays

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

ASF GitHub Bot commented on HIVE-20827:
---------------------------------------

GitHub user pudidic opened a pull request:

    https://github.com/apache/hive/pull/480

    HIVE-20827: Inconsistent results for empty arrays (Teddy Choi)

    Signed-off-by: Teddy Choi <pu...@gmail.com>

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/pudidic/hive HIVE-20827

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/hive/pull/480.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #480
    
----
commit 7b4ef6676dcc103d6149eda0365541cb3ba8cabd
Author: Teddy Choi <pu...@...>
Date:   2018-10-29T22:47:14Z

    HIVE-20827: Inconsistent results for empty arrays (Teddy Choi)
    
    Signed-off-by: Teddy Choi <pu...@gmail.com>

----


> Inconsistent results for empty arrays
> -------------------------------------
>
>                 Key: HIVE-20827
>                 URL: https://issues.apache.org/jira/browse/HIVE-20827
>             Project: Hive
>          Issue Type: Bug
>            Reporter: Teddy Choi
>            Assignee: Teddy Choi
>            Priority: Major
>              Labels: pull-request-available
>         Attachments: HIVE-20827.1.patch
>
>
> LazySimpleDeserializeRead parses an empty array wrong. For example, a line ',' in a text file table with a delimiter ',' and schema 'array<int>, array<array<string>>' shows \[null\], \[\[""\]\], instead of \[\], \[\] with MapReduce engine and vectorized execution enabled. LazySimpleDeserializeRead has following code; 
> {code:java}
> switch (complexField.complexCategory) {
> case LIST:
>   {
>     // Allow for empty string, etc.
>     final boolean isNext = (fieldPosition <= complexFieldEnd);
> {code}
> Empty string value read should be only applied to string families, not to other data types. 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)