You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@arrow.apache.org by "Jorge Leitão (Jira)" <ji...@apache.org> on 2020/12/27 10:59:00 UTC

[jira] [Updated] (ARROW-10508) [Java] Allow FixedSizeListVector to have empty children

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

Jorge Leitão updated ARROW-10508:
---------------------------------
    Component/s: Java

> [Java] Allow FixedSizeListVector to have empty children
> -------------------------------------------------------
>
>                 Key: ARROW-10508
>                 URL: https://issues.apache.org/jira/browse/ARROW-10508
>             Project: Apache Arrow
>          Issue Type: Bug
>          Components: Java
>    Affects Versions: 1.0.0, 2.0.0
>            Reporter: Aleksandr Kopilov
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 3.0.0
>
>          Time Spent: 1h 40m
>  Remaining Estimate: 0h
>
> h3. General problem
> FixedSizeListVector was designed to hold a list of lists with data (filled rectangle matrix) but in real life we can have list of empty lists (zero-width matrix). Java implementation of FixedSizeListVector does not allow to create (and read) such list. But in [specification|http://arrow.apache.org/docs/format/Columnar.html#fixed-size-list-layout] there is no any restriction that listSize can not be zero. Also zero-width matrix can be successfully created by C++ implementation. Java fails on reading it. The most problem is that entire table (VectorSchemaRoot) can not be read if we have at least one FixedSizeListVector.listSize==0 field.
> h3. Expected behavior
>  * Read zero-width matrix (FixedSizeListVector with listSize==0) written by C++ as zero-width matrix.
>  * Read any field from the correct table (written and readable by C++)
> h3. Actual behavior
>  * java.lang.IllegalArgumentException: list size must be positive at org.apache.arrow.util.Preconditions.checkArgument(Preconditions.java:136)
>  * Filled vectors can not be read if at least one fixed list size is 0
> h3. Examples to reproduce
> h4. Reproduce on creating
> Add 
>  {{FixedSizeListVector.empty("ZeroWidthMatrix", 0, new RootAllocator());}} 
>  line of code anywhere
> h4. Reproduce on reading
> [https://github.com/Kopilov/Arrow_FixedSizeListVector_ZeroWidth] repository contains C++ and Java code with GitHub Action workflow that reproduces Actual behavior.
>  Here are some logs: [https://github.com/Kopilov/Arrow_FixedSizeListVector_ZeroWidth/runs/1363206662] (look "Run C++ example", "Build and run Java example")



--
This message was sent by Atlassian Jira
(v8.3.4#803005)