Skip to content

Conversation

vvnginx
Copy link
Contributor

@vvnginx vvnginx commented Aug 20, 2025

Troubleshooting steps for clickhouse running out of memory

Proposed changes

Checklist

Before sharing this pull request, I completed the following checklist:

Footnotes

  1. Potentially sensitive information includes personally identify information (PII), authentication credentials, and live URLs. Refer to the style guide for guidance about placeholder content.

@vvnginx vvnginx requested a review from a team as a code owner August 20, 2025 14:37
@github-actions github-actions bot added documentation Improvements or additions to documentation product/nim Issues related to NGINX Instance Manager labels Aug 20, 2025
Copy link
Contributor

@travisamartin travisamartin left a comment

Choose a reason for hiding this comment

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

This update feels more like configuration and tuning guidance rather than tech spec material. The tech spec guide should stay concise, while detailed instructions like these would be a better fit in the ClickHouse configuration guide:

https://docs.nginx.com/nginx-instance-manager/system-configuration/configure-clickhouse/

Maybe we can move most of this content there, and then leave a short note in the tech spec guide that simply points users to the configuration guide for tuning details.

@vvnginx
Copy link
Contributor Author

vvnginx commented Aug 21, 2025

This update feels more like configuration and tuning guidance rather than tech spec material. The tech spec guide should stay concise, while detailed instructions like these would be a better fit in the ClickHouse configuration guide:

https://docs.nginx.com/nginx-instance-manager/system-configuration/configure-clickhouse/

Maybe we can move most of this content there, and then leave a short note in the tech spec guide that simply points users to the configuration guide for tuning details.

Nice suggestion... incorporated it

@vvnginx vvnginx closed this Aug 21, 2025
@vvnginx vvnginx reopened this Aug 21, 2025
vvnginx and others added 6 commits August 22, 2025 12:26
Click house tuning settings added
Remove from techspecs to clickhouse tuning section
Moving clickhouse to configure-clickhouse section
add a softlink to clickhouse tuning
Corrected reference from tech specs page to clickhouse page
vvnginx and others added 14 commits August 22, 2025 12:26
@travisamartin travisamartin merged commit 8b5d3a7 into nginx:main Aug 22, 2025
7 checks passed
@vvnginx vvnginx deleted the patch-6 branch August 25, 2025 13:36
@vvnginx vvnginx restored the patch-6 branch August 25, 2025 13:37
@vvnginx vvnginx deleted the patch-6 branch August 25, 2025 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation hold for edits product/nim Issues related to NGINX Instance Manager

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants