@getChan I think what we need is to support string type but not modifying the test to other already supported type

D create table date_table(start date, stop date, step interval);
D insert into date_table values (DATE '1992-01-01', DATE '1993-01-02', INTERVAL '1' MONTH);
D select generate_series(start, '1993-03-01', INTERVAL '1 year') from date_table;
┌────────────────────────────────────────────────────────────────────┐
│ generate_series("start", '1993-03-01', CAST('1 year' AS INTERVAL)) │
│                            timestamp[]                             │
├────────────────────────────────────────────────────────────────────┤
│ [1992-01-01 00:00:00, 1993-01-01 00:00:00]                         │
└────────────────────────────────────────────────────────────────────┘

We should able to return the same result as Duckdb's

In that case, I guess we should extend the above issue to "Add string type support to generate_series function".

I expect you get the same answer as if you explicitly cast the second argument to date:

The goal is make the query return the same result as the one you explicitly cast the argument.

In that case, I guess we should extend the above issue to "Add string type support to generate_series function".

Yes, I think what we need is to support string type, or cast the string to date. Not sure what is the exact solution. But I expect we should get the correct result for that query

Yes, I think what we need is to support string type, or cast the string to date. Not sure what is the exact solution. But I expect we should get the correct result for that query

The type coercsion system is supposed to handle adding this kind of cast.

I wonder if we can could remove the Any(3) signature a new signature to generate_series :

I think that would then ensure DataFusion actually casts the arguments to the supported types

Marking as draft as I think this PR is no longer waiting on feedback. Please mark it as ready for review when it is ready for another look

I wonder if we can could remove the Any(3) signature a new signature to generate_series :

removed Any(3) signature. and fix some type coerce code for range, generate_series