[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#63926] [PATCH] services: postgresql: Add more role fields.
From: |
Ludovic Courtès |
Subject: |
[bug#63926] [PATCH] services: postgresql: Add more role fields. |
Date: |
Wed, 14 Jun 2023 12:38:36 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
Hi,
(Cc: Christopher Baines who knows pgsql better than I do.)
Miguel Ángel Moreno <me@mianmoreno.com> skribis:
> * gnu/services/databases.scm (postgresql-role): Add more role fields.
> (postgresql-create-roles): Honor it.
> * doc/guix.texi (Database Services): Document it.
That looks like an improvement! Some comments:
> @item @code{create-database?} (default: @code{#f})
> -Whether to create a database with the same name as the role.
> +whether to create a database with the same name as the role.
This change is unnecessary (leftover?).
> +@item @code{encoding} (default: @code{"UTF8"})
> +The character set to use for storing text in the database.
Or "UTF-8"? It might be worth mentioning the naming convention for
encodings, especially if it differs from
<http://www.iana.org/assignments/character-sets>.
> +@item @code{collation} (default: @code{"en_US.utf8"})
> +The string sort order locale setting.
> +
> +@item @code{ctype} (default: @code{"en_US.utf8"})
> +The character classification locale setting.
> +
> +@item @code{template} (default: @code{"template1"})
> +The default template to copy the new database from when creating it.
> +Use @code{"template0"} for a pristine database with no system-local
> +modifications.
Are these defaults equivalent to those that were previously in effect?
If would be ideal.
Thanks, and apologies for the delay!
Ludo’.