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.

Sign up for GitHub

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

Jump to bottom

Temporal: support date/time literals? #520

Closed
seancorfield opened this issue Jan 2, 2024 · 1 comment
Closed

Temporal: support date/time literals? #520

seancorfield opened this issue Jan 2, 2024 · 1 comment
Assignees
@seancorfield
Labels
enhancement

Comments

@seancorfield
Copy link
Owner

DATE '2024-01-01' etc.

Is it worth having specific syntax for this, or just assume folks will use ? and pass an #inst parameter?
@seancorfield seancorfield added the needs analysis I need to think about this! label Jan 2, 2024
@seancorfield seancorfield self-assigned this Jan 2, 2024
@seancorfield
Copy link
Owner Author

Since DATE() and TIME() are valid functions in some dialects, we cannot special-case them as literals, therefore we need some additional syntax.

Expanding :inline to support multiple arguments is probably the easiest route to this: [:inline :DATE "2024-01-06"]

@seancorfield seancorfield added enhancement and removed needs analysis I need to think about this! labels Jan 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@seancorfield seancorfield

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@seancorfield