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

docs: Update postgreSQL docs #9586

Closed
wants to merge 3 commits into from
Closed

Conversation

tara-hpe
Copy link
Contributor

@tara-hpe tara-hpe commented Jun 27, 2024

Replaced by

#9594

Description

audience: admin
purpose: users often miss important information including the recommended configuration for max_connections and other settings for bigger deployments

since this links from the "advanced installation" guide it is appropriate to move the content to its own page

@cla-bot cla-bot bot added the cla-signed label Jun 27, 2024
@tara-hpe tara-hpe requested a review from mackrorysd June 27, 2024 17:58
@determined-ci determined-ci requested a review from a team June 27, 2024 17:58
@determined-ci determined-ci added the documentation Improvements or additions to documentation label Jun 27, 2024
Copy link

netlify bot commented Jun 27, 2024

Deploy Preview for determined-ui ready!

Name Link
🔨 Latest commit 2adca7c
🔍 Latest deploy log https://app.netlify.com/sites/determined-ui/deploys/667dddc937b43100081f62ed
😎 Deploy Preview https://deploy-preview-9586--determined-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@tara-hpe tara-hpe force-pushed the docs/Update-postgreSQL-docs branch from 85dd6b8 to 1dee4b8 Compare June 27, 2024 21:30
@tara-hpe tara-hpe force-pushed the docs/Update-postgreSQL-docs branch from 6df8636 to 2adca7c Compare June 27, 2024 21:46
max_connections
*****************

``max_connections`` is the most important and universal parameter to adjust. If you are using an
Copy link
Member

Choose a reason for hiding this comment

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

I would drop "if you are using an existing PostgreSQL installation". Even on a new installation, depending on how you get and install Postgres, the default may be too low.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

thanks! makes sense.

@tara-hpe tara-hpe closed this Jul 1, 2024
tara-hpe added a commit that referenced this pull request Jul 1, 2024
replaces: #9586

audience: admin
purpose: users often miss important information including the recommended configuration for max_connections and other settings for bigger deployments

since this links from the "advanced installation" guide it is appropriate to move the content to its own page
@tara-hpe tara-hpe mentioned this pull request Jul 1, 2024
5 tasks
tara-hpe added a commit that referenced this pull request Jul 1, 2024
replaces: #9586

audience: admin
purpose: users often miss important information including the recommended configuration for max_connections and other settings for bigger deployments

since this links from the "advanced installation" guide it is appropriate to move the content to its own page
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants