apache / apache/datafusion-comet

Change UDF signature to use ColumnarValue rather than raw Arrow types

Open
#4,358 1 comment 0 reactions 0 assignees View on GitHub
area:expressions enhancement priority:low
Dominant language
Scala
Stars
1.3k
Forks
373
Avg merge
2d 6h
Merged PRs (30d)
190

Description

### What is the problem the feature request solves?

### Is your feature request related to a problem or challenge?

DataFusion uses `ColumnarValue` to represent Arrays and Scalars.

```rust
#[derive(Clone, Debug)]
pub enum ColumnarValue {
/// Array of values
Array(ArrayRef),
/// A single value
Scalar(ScalarValue),
}
```

Perhaps we should use this in the UDF signatures. This avoids the need to pass in row counts.

### Describe the solution you'd like

_No response_

### Describe alternatives you've considered

_No response_

### Additional context

_No response_

### Describe the potential solution

_No response_

### Additional context

_No response_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.