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

Updated scaling.md for revised scaling docs #1516

Merged
merged 3 commits into from
Sep 18, 2023
Merged

Conversation

aashishkohli
Copy link
Contributor

Updated autoscaling docs

@CLAassistant
Copy link

CLAassistant commented Sep 12, 2023

CLA assistant check
All committers have signed the CLA.

@robot-ch-test-poll
Copy link

robot-ch-test-poll commented Sep 12, 2023

This is an automated comment for commit ff1ee91 with description of existing statuses. It's updated for the latest CI running

❌ Click here to open a full report in a separate page

Check nameDescriptionStatus
Docs CheckBuilds and tests the documentation❌ failure
VercelThere's no description for the check yet, please add it to tests/ci/ci_config.py:CHECK_DESCRIPTIONS⏳ pending

Made minor edits.
@tbragin
Copy link
Member

tbragin commented Sep 15, 2023

Although I was not listed as a reviewer, I took a look and it appears to be an improvement. Thank you for working on this @aashishkohli !

[ClickHouse Cloud API](/docs/en/cloud/manage/api/services-api-reference.md) to preemptively scale up your service to handle the spike and scale it down once the demand subsides. To understand current service size and the number of replicas you can run the query below

```
SELECT hostname(), getSetting('max_threads'), formatReadableSize(getSetting('max_memory_usage'))
Copy link
Member

Choose a reason for hiding this comment

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

Should be replace by:

SELECT *
FROM clusterAllReplicas('default', view(
    SELECT
        hostname() AS server,
        getSetting('max_threads') as cpu_cores,
         formatReadableSize(getSetting('max_memory_usage')) as memory
    FROM system.one
))
ORDER BY server ASC
SETTINGS skip_unavailable_shards = 1

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks! I'll update this

@justindeguzman justindeguzman merged commit 22df0fc into main Sep 18, 2023
2 of 3 checks passed
@justindeguzman justindeguzman deleted the aashishkohli-patch-1 branch September 18, 2023 04:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants