Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support select range from datetime #9353

Closed
Lordworms opened this issue Feb 27, 2024 · 5 comments · Fixed by #9420
Closed

Support select range from datetime #9353

Lordworms opened this issue Feb 27, 2024 · 5 comments · Fixed by #9420
Assignees
Labels
enhancement New feature or request

Comments

@Lordworms
Copy link
Contributor

Is your feature request related to a problem or challenge?

image duckdb did it image

Describe the solution you'd like

No response

Describe alternatives you've considered

No response

Additional context

No response

@Lordworms Lordworms added the enhancement New feature or request label Feb 27, 2024
@Lordworms
Copy link
Contributor Author

take

@Lordworms
Copy link
Contributor Author

wait until #9352 merged

@Omega359
Copy link
Contributor

Is the range function still going to be an alias of generate_series but that generate_series will now support date/time intervals such as (postgresql syntax) ?

SELECT generate_series('2012-06-29', '2012-07-03', '1 day'::interval);

@Lordworms
Copy link
Contributor Author

Is the range function still going to be an alias of generate_series but that generate_series will now support date/time intervals such as (postgresql syntax) ?

SELECT generate_series('2012-06-29', '2012-07-03', '1 day'::interval);

No, with #9352 I have separated the range and generate_series since they stands for different meaning in duckDB, and right now I am going to support the syntax in duckDB for datetime

@Omega359
Copy link
Contributor

My understanding is that at this time DataFusion wants to maintain compatibility with postgresql syntax as much as possible which is why I was asking. That being said postgresql doesn't have a range function exactly (what they have is pretty weird tbh) so having a range function is a nice addition. I would have looked at having generate_series though support an optional 3rd param for interval though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants