Skip to content

Nullability of array_agg #11094

@eejbyfeldt

Description

@eejbyfeldt

Describe the bug

The logic for nullability for the return value of array_agg is currently correct.

To Reproduce

The current logic is here:

fn field(&self) -> Result<Field> {
Ok(Field::new_list(
&self.name,
// This should be the same as return type of AggregateFunction::ArrayAgg
Field::new("item", self.input_data_type.clone(), true),
self.nullable,
))
}

it returns a list where the nullability depens on the input and the field in the list is always nullable.

Expected behavior

The fields nullability should depend on the input_nullability and the nullability of the list should be false.

Additional context

No response

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions