Skip to content

Postgres Tuner - Added a parameter #6858

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

Merged

Conversation

nidhibhammar
Copy link
Contributor

@nidhibhammar nidhibhammar commented May 29, 2025

What Changed?

Added a parameter as per DOCS-1555

@nidhibhammar nidhibhammar requested a review from a team as a code owner May 29, 2025 11:49
@nidhibhammar nidhibhammar added the deploy Add this label to a PR and it will automatically be deployed to netlify label May 29, 2025
Copy link
Contributor

github-actions bot commented May 29, 2025

@github-actions github-actions bot temporarily deployed to pull request May 29, 2025 11:53 Inactive
@nidhibhammar nidhibhammar requested a review from tusharahuja May 30, 2025 07:21
Copy link
Contributor

@tusharahuja tusharahuja left a comment

Choose a reason for hiding this comment

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

Thanks, looks good, can we also add the default parameter location file name i.e

edb=# show edb_pg_tuner.container_memory_limit_file_path;
 edb_pg_tuner.container_memory_limit_file_path
-----------------------------------------------
 /sys/fs/cgroup/memory/memory.limit_in_bytes
(1 row)

@nidhibhammar nidhibhammar requested a review from tusharahuja June 5, 2025 09:00
@github-actions github-actions bot temporarily deployed to pull request June 5, 2025 09:03 Inactive
Copy link
Contributor

@tusharahuja tusharahuja left a comment

Choose a reason for hiding this comment

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

looks good. Thanks.

Copy link
Contributor

@djw-m djw-m left a comment

Choose a reason for hiding this comment

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

LGTM

@nidhibhammar nidhibhammar merged commit 760679a into develop Jun 9, 2025
8 checks passed
@nidhibhammar nidhibhammar deleted the content/docs/postgres_tuner/add_parameter/docs-1555 branch June 9, 2025 09:39
@djw-m djw-m mentioned this pull request Jun 10, 2025
edb-slonik pushed a commit that referenced this pull request Jul 1, 2025
Small edit to American spelling
edb-slonik pushed a commit that referenced this pull request Jul 1, 2025
Edits to Postgres Tuner - Added a parameter #6858
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deploy Add this label to a PR and it will automatically be deployed to netlify
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants