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

Add perf test for time zone operators #10260

Merged
merged 1 commit into from
Jan 29, 2024
Merged

Conversation

res-life
Copy link
Collaborator

@res-life res-life commented Jan 24, 2024

closes #10142

Test result:

operator tz mean cpu mean gpu ratio
from_utc_timestamp Asia/Shanghai 689.20 ms 111.00 ms 6.21 x
from_utc_timestamp Japan 608.00 ms 92.60 ms 6.57 x
to_utc_timestamp Asia/Shanghai 811.20 ms 84.80 ms 9.57 x
to_utc_timestamp Japan 737.40 ms 82.60 ms 8.93 x
hour Asia/Shanghai 565.80 ms 83.40 ms 6.78 x
hour Japan 520.60 ms 77.80 ms 6.69 x
minute Asia/Shanghai 546.80 ms 75.40 ms 7.25 x
minute Japan 510.60 ms 73.40 ms 6.96 x
second Asia/Shanghai 548.80 ms 74.20 ms 7.40 x
second Japan 517.60 ms 70.40 ms 7.35 x
unix_timestamp Asia/Shanghai 3439.00 ms 76.20 ms 45.13 x
unix_timestamp Japan 3410.20 ms 75.60 ms 45.11 x
from_unixtime Asia/Shanghai 3494.40 ms 81.20 ms 43.03 x
from_unixtime Japan 3309.80 ms 76.00 ms 43.55 x
date_format Asia/Shanghai 3399.60 ms 72.60 ms 46.83 x
date_format Japan 3373.80 ms 73.40 ms 45.96 x

Signed-off-by: Chong Gao res_life@163.com

Signed-off-by: Chong Gao <res_life@163.com>
@res-life
Copy link
Collaborator Author

build

@res-life res-life self-assigned this Jan 24, 2024
@res-life
Copy link
Collaborator Author

build

Copy link
Collaborator

@thirtiseven thirtiseven left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sameerz sameerz added test Only impacts tests performance A performance related task/issue labels Jan 26, 2024
@res-life
Copy link
Collaborator Author

build

@res-life res-life merged commit de59e2e into NVIDIA:branch-24.02 Jan 29, 2024
40 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
performance A performance related task/issue test Only impacts tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[TASK] Benchmark existing timestamp functions that work in non-UTC time zone (non-DST)
3 participants