jonahgao closed issue #13872: Functionality of `array_repeat` udf
URL: https://github.com/apache/datafusion/issues/13872
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsu
jonahgao commented on issue #13872:
URL: https://github.com/apache/datafusion/issues/13872#issuecomment-2579050717
It seems to be a display/formatting issue for me. We should always display
NULLs within containers, like Postgres and DuckDB do.
```psql
psql=> select ARRAY[null];
arr
jatin510 commented on issue #13872:
URL: https://github.com/apache/datafusion/issues/13872#issuecomment-2573601983
@jayzhan211 i will work on the issue for `array_repeat`
in case if the array is empty, for the `array_repeat` the ouptut will be
`null` intead of `empty array`
will
jayzhan211 commented on issue #13872:
URL: https://github.com/apache/datafusion/issues/13872#issuecomment-2569907599
Which one do you think should return null?
> SELECT cardinality(array_remove([1], 1));
I think we should return 0 for this case 🤔 ?
--
This is an automated
jatin510 commented on issue #13872:
URL: https://github.com/apache/datafusion/issues/13872#issuecomment-2569547645
```
> SELECT array_remove([1,null], 1);
+--+
| array_remove(make_array(Int64(1),NULL),Int64(1)) |
+---
jayzhan211 commented on issue #13872:
URL: https://github.com/apache/datafusion/issues/13872#issuecomment-2567586060
> > I think we can return null for this case
>
> should we make this change ?
> @jayzhan211 @alamb
It makes sense to me
--
This is an automated message f
jatin510 commented on issue #13872:
URL: https://github.com/apache/datafusion/issues/13872#issuecomment-2567574580
> I think we can return null for this case
should we make this change ?
@jayzhan211 @alamb
--
This is an automated message from the Apache Git Service.
To respond
jatin510 commented on issue #13872:
URL: https://github.com/apache/datafusion/issues/13872#issuecomment-2560679722
> I think we can return null for this case
This will be a breaking change.
@jayzhan211
cc: @alamb
--
This is an automated message from the Apache
jayzhan211 commented on issue #13872:
URL: https://github.com/apache/datafusion/issues/13872#issuecomment-2558290456
empty array `[]` and array of null with len = 1 `[]` looks the same but they
are different.
> SELECT array_repeat(ARRAY[1, 2], null)
I think we can return null
jatin510 opened a new issue, #13872:
URL: https://github.com/apache/datafusion/issues/13872
### Describe the bug
How `array_repeat` works currently for `null` as input in array or key:
```
> SELECT array_repeat(null,0 ) AS repeated_array;
++
| repeated_a
10 matches
Mail list logo
