-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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
Faster startup of clickhouse-client and clickhouse-local #52489
Conversation
This is an automated comment for commit 75efee9 with description of existing statuses. It's updated for the latest CI running
|
@alexey-milovidov have you seen significant different? I've tested it and do not see it: azat@s1:~/ch/tmp/52228$ time ./clickhouse-52228-debug local -q 'select 1'
1
real 0m0.926s
user 0m0.876s
sys 0m0.064s
azat@s1:~/ch/tmp/52228$ time ./clickhouse-pre-52228-debug local -q 'select 1'
1
real 0m0.920s
user 0m0.894s
sys 0m0.036s
azat@s1:~/ch/tmp/52228$ time ./clickhouse-52228 local -q 'select 1'
1
real 0m0.074s
user 0m0.035s
sys 0m0.043s
azat@s1:~/ch/tmp/52228$ time ./clickhouse-pre-52228 local -q 'select 1'
1
real 0m0.074s
user 0m0.050s
sys 0m0.026s Looks like it does not makes a lot of difference since And the last time I looked at this I saw the the most time takes DateLUT to pre fill the tables. |
Only when #47475 is enabled. |
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Improve the startup time of
clickhouse-client
andclickhouse-local
in debug and sanitizer builds. This closes #52228.