I would vote for treating nulls as empty. On Fri, Jan 10, 2020 at 12:36 AM Ji Liu <niki...@aliyun.com.invalid> wrote:
> Hi all, > > Currently isEmpty API is always return false in BaseRepeatedValueVector, > and its subclass ListVector did not overwrite this method. > This will lead to incorrect result, for example, a ListVector with data > [1,2], null, [], [5,6] would get [false, false, false, false] which is not > right. > I opened a PR to fix this[1] and not sure what’s the right behavior for > null value, should it return [false, false, true, false] or [false, true, > true, false] ? > > > Thanks, > Ji Liu > > > [1] https://github.com/apache/arrow/pull/6044 > >
