-
Notifications
You must be signed in to change notification settings - Fork 458
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
[GLUTEN-7100][CH] support function timestamp_seconds/timestamp_millis/timestamp_micros #7102
Conversation
Run Gluten Clickhouse CI |
Run Gluten Clickhouse CI |
cf228a7
to
ae494af
Compare
Run Gluten Clickhouse CI |
1 similar comment
Run Gluten Clickhouse CI |
2879cad
to
b93718a
Compare
Run Gluten Clickhouse CI |
Run Gluten Clickhouse CI |
2 uts are excluded because of #7169
|
Run Gluten Clickhouse CI |
8282bae
to
fec0eb4
Compare
Run Gluten Clickhouse CI |
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.
LGTM
Run Gluten Clickhouse CI |
4af372f
to
54ac85e
Compare
Run Gluten Clickhouse CI |
54ac85e
to
6b2aaa4
Compare
Run Gluten Clickhouse CI |
6b2aaa4
to
f01e704
Compare
Run Gluten Clickhouse CI |
…/timestamp_micros (apache#7102) * support function timestamp_xxx * fix style * disable ut TIMESTAMP_SECONDS because of ClickHouse/ClickHouse#69280 * fix all failed uts * fix failed uts
What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
(Fixes: #7100)
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)