Re: Naming of network_ops vs. inet_ops for SP-GIST

2023-01-24 Thread Tom Lane
Michael Paquier writes: > I don't have a strong opinion about the naming inconsistency between > the opclass name and the opfamily name in this case, though, couldn't > it create more problems than actually fix something? Well, it's been like that from day one and people haven't complained. I

log_temp_files one minor clarification

2023-01-24 Thread Martin L. Buchanan
https://www.postgresql.org/docs/current/runtime-config-logging.html log_temp_files (integer) Controls logging of temporary file names and sizes. Temporary files can be created for sorts, hashes, and temporary query results. If enabled by this setting, a log entry is emitted for each temporary

Re: Naming of network_ops vs. inet_ops for SP-GIST

2023-01-24 Thread Michael Paquier
On Tue, Jan 24, 2023 at 03:22:44PM -0500, Tom Lane wrote: > I wonder whether we shouldn't just revert this table to > showing opclass names, and avert our eyes from the theoretical > inconsistency. Michael, looks like it was your 7a1cd5260 > that changed it; what do you think? Yes, the docs

Re: Naming of network_ops vs. inet_ops for SP-GIST

2023-01-24 Thread Laurence Parry
> This naming was evidently chosen to match btree, > which has both inet_ops and cidr_ops opclasses > within its network_ops family. > spgist only supports inet_ops FWIW, the documentation for GIST has inet_ops in the equivalent table, so it was extra-confusing because I thought SP-GIST's

Re: Naming of network_ops vs. inet_ops for SP-GIST

2023-01-24 Thread Tom Lane
PG Doc comments form writes: > I wanted to add an SP-GIST index for an inet field ip_address > In https://www.postgresql.org/docs/14/spgist-builtin-opclasses.html > network_ops is stated as the built-in opclass for (inet, inet) > I entered the following command: > # create index concurrently

Re: Naming of network_ops vs. inet_ops for SP-GIST

2023-01-24 Thread David G. Johnston
Michael, please see below: On Tue, Jan 24, 2023 at 12:44 PM PG Doc comments form < nore...@postgresql.org> wrote: > The following documentation comment has been logged on the website: > > Page: https://www.postgresql.org/docs/14/spgist-builtin-opclasses.html > Description: > > I wanted to add an

Re: gen_random_uuid is only available with pgcrypto enabled

2023-01-24 Thread David G. Johnston
On Tue, Jan 24, 2023 at 12:43 PM PG Doc comments form < nore...@postgresql.org> wrote: > The following documentation comment has been logged on the website: > > Page: https://www.postgresql.org/docs/15/functions-uuid.html > Description: > > Regarding the documentation on this page: >

Re: gen_random_uuid is only available with pgcrypto enabled

2023-01-24 Thread Tom Lane
PG Doc comments form writes: > Regarding the documentation on this page: > https://www.postgresql.org/docs/15/functions-uuid.html > It think the documentation should state clearly, that postgres natively does > not have any UUID generation functions. That hasn't been true since we added

Re: https://www.postgresql.org/account/comments/new/15/sql-alterdatabase.html/

2023-01-24 Thread David G. Johnston
On Tue, Jan 24, 2023 at 12:43 PM PG Doc comments form < nore...@postgresql.org> wrote: > The following documentation comment has been logged on the website: > > Page: https://www.postgresql.org/docs/15/sql-alterdatabase.html > Description: > > Would love to see more information about each of the

Naming of network_ops vs. inet_ops for SP-GIST

2023-01-24 Thread PG Doc comments form
The following documentation comment has been logged on the website: Page: https://www.postgresql.org/docs/14/spgist-builtin-opclasses.html Description: I wanted to add an SP-GIST index for an inet field ip_address In https://www.postgresql.org/docs/14/spgist-builtin-opclasses.html network_ops

https://www.postgresql.org/account/comments/new/15/sql-alterdatabase.html/

2023-01-24 Thread PG Doc comments form
The following documentation comment has been logged on the website: Page: https://www.postgresql.org/docs/15/sql-alterdatabase.html Description: Would love to see more information about each of the ALTER DATABASE parameters... a specific... "connlimit"... -1 means no connections, but what does 0

gen_random_uuid is only available with pgcrypto enabled

2023-01-24 Thread PG Doc comments form
The following documentation comment has been logged on the website: Page: https://www.postgresql.org/docs/15/functions-uuid.html Description: Regarding the documentation on this page: https://www.postgresql.org/docs/15/functions-uuid.html It think the documentation should state clearly, that