public inbox for [email protected]
help / color / mirror / Atom feedFrom: Michael Paquier <[email protected]>
To: Robert Haas <[email protected]>
Cc: Postgres hackers <[email protected]>
Subject: Re: pgsql: Add support for building with ZSTD.
Date: Mon, 21 Feb 2022 10:02:58 +0900
Message-ID: <[email protected]> (raw)
In-Reply-To: <CA+TgmoZLtMURjevDpzGQyW46GUdTJNRemNujxkBQR_KMuOTKMg@mail.gmail.com>
References: <[email protected]>
<YhCL0fKnDv/[email protected]>
<CA+TgmoZLtMURjevDpzGQyW46GUdTJNRemNujxkBQR_KMuOTKMg@mail.gmail.com>
On Sun, Feb 20, 2022 at 09:45:05AM -0500, Robert Haas wrote:
> No issues at all with you adjusting this, but I think that sentence
> reads a little awkwardly.
Thanks.
> Perhaps instead of "The default is
> <literal>zstd</literal>, that would be the command found in
> <varname>PATH</varname>." you could write something like "The default
> is <literal>zstd</literal>, which will search for a command by that
> name in the configured <varname>PATH</varname>." Or maybe something
> else is better, not sure exactly, your version just seems a little odd
> to me.
Okay, done then. We've been using the same wording for all the other
variables, and what you are suggesting here sounds much better to me,
so I have adjusted all the descriptions this way, and added the ZSTD
part.
--
Michael
Attachments:
[application/pgp-signature] signature.asc (833B, ../[email protected]/2-signature.asc)
download
view thread (2+ messages)
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected], [email protected]
Subject: Re: pgsql: Add support for building with ZSTD.
In-Reply-To: <[email protected]>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox