goldmedal commented on issue #13371:
URL: https://github.com/apache/datafusion/issues/13371#issuecomment-2474008866
I prefer to provide a real type name here instead of a semantic name (e.g.
`string`, `int`). The similar behavior in DuckDB like
```
D select function_name, parameter_types from duckdb_functions() where
function_name = 'range';
┌───────────────┬────────────────────────────────────────────────────────────────┐
│ function_name │ parameter_types
│
│ varchar │ varchar[]
│
├───────────────┼────────────────────────────────────────────────────────────────┤
│ range │ [BIGINT]
│
│ range │ [BIGINT, BIGINT]
│
│ range │ [BIGINT, BIGINT, BIGINT]
│
│ range │ [TIMESTAMP, TIMESTAMP, INTERVAL]
│
│ range │ [TIMESTAMP WITH TIME ZONE, TIMESTAMP WITH TIME ZONE,
INTERVAL] │
│ range │ [BIGINT]
│
│ range │ [BIGINT, BIGINT]
│
│ range │ [BIGINT, BIGINT, BIGINT]
│
│ range │ [TIMESTAMP, TIMESTAMP, INTERVAL]
│
│ range │ [TIMESTAMP WITH TIME ZONE, TIMESTAMP WITH TIME ZONE,
INTERVAL] │
├───────────────┴────────────────────────────────────────────────────────────────┤
│ 10 rows 2
columns │
└────────────────────────────────────────────────────────────────────────────────┘
```
Because we have the logic type now, maybe showing the logic type name would
be a better choice. WDYT?
https://github.com/apache/datafusion/blob/main/datafusion/common/src/types/native.rs
--
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 unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org
