-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Add today
alias for current_date
#8423
Conversation
2fea7a8
to
56b5373
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @smallzhongfeng for this PR
Please add also tests and user documentation scalar_functions.md
current_date
today
alias for current_date
Thanks. Updated. |
Please add Also the tests are needed, we can likely use |
I add a test, because the function |
true | ||
|
||
query B | ||
select now() = current_date(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you add it for today
as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @smallzhongfeng
Thanks @smallzhongfeng |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you again @smallzhongfeng -- this looks great
Co-authored-by: Alex Huang <[email protected]>
Thank you @Weijun-H |
Which issue does this PR close?
Closes #.
Rationale for this change
ck support udf
today
https://clickhouse.com/docs/en/sql-reference/functions/date-time-functions#:~:text=Accepts%20zero%20arguments%20and%20returns%20the%20current%20date%20at%20one%20of%20the%20moments%20of%20query%20analysis.%20The%20same%20as%20%E2%80%98toDate(now())%E2%80%99.duckdb https://duckdb.org/docs/archive/0.9.2/sql/functions/date#:~:text=1992%2D04%2D01-,today(),-Current%20date%20(start
What changes are included in this PR?
Add a new name for
current_date
Are these changes tested?
Are there any user-facing changes?