Skip to content

Conversation

@bobjolliffe
Copy link
Collaborator

Made a few tuning changes

Modified README to clarify the way config settings are applied to a running postgresql server
@bobjolliffe bobjolliffe requested a review from tonsV2 December 16, 2025 21:47
@tonsV2 tonsV2 changed the title Postgres tuning revision feat: Postgres tuning revision Dec 17, 2025
Custom configuration for Postgresql can be done by adding to the files in the `./config/postgresql/conf.d/` directory. If your configuration doesn't belong in either of the existing files, you can create a new file. However, it's advised not to make changes to the [postgresql.conf](config/postgresql/postgresql.conf) file.

## Deployment For Production
Some postgresql configuration parameters can be applied by reloading the configuration. Others will only take effect when the database is restarted. The detailed description of each configuration option (on postgresql v16) is avalable [here](https://www.postgresql.org/docs/16/runtime-config.html). The documentation for each setting generally indicates whether the setting requires a restart in order to take effect.
Copy link
Collaborator

Choose a reason for hiding this comment

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

You have multiple double spaces in this paragraph

```

### Deployment Prerequisites
If you need to restart the database server, it is safest to stop the application container first, restart the database, then start the application again. This will mean some minutes of downtime for your users so it is best to co-ordinate such operations with users or execute during quiet times of the day. A typical sequence might look like:
Copy link
Collaborator

Choose a reason for hiding this comment

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

You have multiple double spaces in this paragraph

@tonsV2 tonsV2 requested a review from radnov December 17, 2025 03:21
changed postgres restart instructions as ber suggestion

Co-authored-by: tonsV2 <tonsV2@users.noreply.github.com>
Copy link
Contributor

@Philip-Larsen-Donnelly Philip-Larsen-Donnelly left a comment

Choose a reason for hiding this comment

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

The additions look sensible, but unfortunately this PR branch is based on quite an old version of the README.md, so it can't be merged as is.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants