public inbox for [email protected]
help / color / mirror / Atom feed[PATCH v6 05/10] doc: s/evade/avoid/
7+ messages / 5 participants
[nested] [flat]
* [PATCH v6 05/10] doc: s/evade/avoid/
@ 2020-04-06 22:16 Justin Pryzby <[email protected]>
0 siblings, 0 replies; 7+ messages in thread
From: Justin Pryzby @ 2020-04-06 22:16 UTC (permalink / raw)
---
src/backend/access/gin/README | 2 +-
src/backend/utils/adt/jsonpath_exec.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/backend/access/gin/README b/src/backend/access/gin/README
index 125a82219b..41d4e1e8a0 100644
--- a/src/backend/access/gin/README
+++ b/src/backend/access/gin/README
@@ -413,7 +413,7 @@ leftmost leaf of the tree.
Deletion algorithm keeps exclusive locks on left siblings of pages comprising
currently investigated path. Thus, if current page is to be removed, all
required pages to remove both downlink and rightlink are already locked. That
-evades potential right to left page locking order, which could deadlock with
+avoids potential right to left page locking order, which could deadlock with
concurrent stepping right.
A search concurrent to page deletion might already have read a pointer to the
diff --git a/src/backend/utils/adt/jsonpath_exec.c b/src/backend/utils/adt/jsonpath_exec.c
index f146767bfc..2d2eb7d7a3 100644
--- a/src/backend/utils/adt/jsonpath_exec.c
+++ b/src/backend/utils/adt/jsonpath_exec.c
@@ -35,7 +35,7 @@
* executeItemOptUnwrapTarget() function have 'unwrap' argument, which indicates
* whether unwrapping of array is needed. When unwrap == true, each of array
* members is passed to executeItemOptUnwrapTarget() again but with unwrap == false
- * in order to evade subsequent array unwrapping.
+ * in order to avoid subsequent array unwrapping.
*
* All boolean expressions (predicates) are evaluated by executeBoolItem()
* function, which returns tri-state JsonPathBool. When error is occurred
--
2.17.0
--C+ts3FVlLX8+P6JN
Content-Type: text/x-diff; charset=us-ascii
Content-Disposition: attachment;
filename="v6-0006-doc-Add-logical-replication-support-to-replicate-.patch"
^ permalink raw reply [nested|flat] 7+ messages in thread
* Re: typos
@ 2022-04-20 21:32 Alvaro Herrera <[email protected]>
0 siblings, 2 replies; 7+ messages in thread
From: Alvaro Herrera @ 2022-04-20 21:32 UTC (permalink / raw)
To: Justin Pryzby <[email protected]>; +Cc: David Rowley <[email protected]>; pgsql-hackers; Fabien COELHO <[email protected]>; Amit Kapila <[email protected]>
On 2022-Apr-19, Alvaro Herrera wrote:
> I propose we standardize on <productname>Zstd</productname> everywhere.
> Users can look it up if they're really interested.
So the attached.
There are other uses of <literal>zstd</literal>, but those are referring to the
executable program.
--
Álvaro Herrera PostgreSQL Developer — https://www.EnterpriseDB.com/
"No necesitamos banderas
No reconocemos fronteras" (Jorge González)
Attachments:
[text/x-diff] 0001-Zstd.patch (2.3K, ../../[email protected]/2-0001-Zstd.patch)
download | inline diff:
From 256904cc5a8718ac081dbf51f6263ab022e503f6 Mon Sep 17 00:00:00 2001
From: Alvaro Herrera <[email protected]>
Date: Wed, 20 Apr 2022 23:26:28 +0200
Subject: [PATCH] Zstd
---
doc/src/sgml/install-windows.sgml | 6 +++---
doc/src/sgml/installation.sgml | 4 ++--
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/doc/src/sgml/install-windows.sgml b/doc/src/sgml/install-windows.sgml
index 43cc5f6f5b..104670d295 100644
--- a/doc/src/sgml/install-windows.sgml
+++ b/doc/src/sgml/install-windows.sgml
@@ -307,9 +307,9 @@ $ENV{MSBFLAGS}="/m";
</varlistentry>
<varlistentry>
- <term><productname>ZSTD</productname></term>
+ <term><productname>Zstd</productname></term>
<listitem><para>
- Required for supporting <productname>ZSTD</productname> compression
+ Required for supporting <productname>Zstd</productname> compression
method. Binaries and source can be downloaded from
<ulink url="https://github.com/facebook/zstd/releases"></ulink>.
</para></listitem>
@@ -560,7 +560,7 @@ $ENV{PROVE_TESTS}='t/020*.pl t/010*.pl'
<varlistentry>
<term><varname>ZSTD</varname></term>
<listitem><para>
- Path to a <application>zstd</application> command. The default is
+ Path to a <application>Zstd</application> command. The default is
<literal>zstd</literal>, which will search for a command by that
name in the configured <envar>PATH</envar>.
</para></listitem>
diff --git a/doc/src/sgml/installation.sgml b/doc/src/sgml/installation.sgml
index df32025a86..c60484e221 100644
--- a/doc/src/sgml/installation.sgml
+++ b/doc/src/sgml/installation.sgml
@@ -273,7 +273,7 @@ su - postgres
<listitem>
<para>
- You need <productname>zstd</productname>, if you want to support
+ You need <productname>Zstd</productname>, if you want to support
compression of data with that method; see
<xref linkend="guc-wal-compression"/>.
The minimum required version is 1.4.0.
@@ -996,7 +996,7 @@ build-postgresql:
<term><option>--with-zstd</option></term>
<listitem>
<para>
- Build with <productname>ZSTD</productname> compression support.
+ Build with <productname>Zstd</productname> compression support.
</para>
</listitem>
</varlistentry>
--
2.30.2
^ permalink raw reply [nested|flat] 7+ messages in thread
* Re: typos
@ 2022-04-20 21:38 Justin Pryzby <[email protected]>
parent: Alvaro Herrera <[email protected]>
1 sibling, 1 reply; 7+ messages in thread
From: Justin Pryzby @ 2022-04-20 21:38 UTC (permalink / raw)
To: Alvaro Herrera <[email protected]>; +Cc: David Rowley <[email protected]>; pgsql-hackers; Fabien COELHO <[email protected]>; Amit Kapila <[email protected]>
On Wed, Apr 20, 2022 at 11:32:08PM +0200, Alvaro Herrera wrote:
> On 2022-Apr-19, Alvaro Herrera wrote:
>
> > I propose we standardize on <productname>Zstd</productname> everywhere.
> > Users can look it up if they're really interested.
>
> So the attached.
>
> There are other uses of <literal>zstd</literal>, but those are referring to the
> executable program.
This one shouldn't be changed, or not like this?
> @@ -560,7 +560,7 @@ $ENV{PROVE_TESTS}='t/020*.pl t/010*.pl'
> <varlistentry>
> <term><varname>ZSTD</varname></term>
> <listitem><para>
> - Path to a <application>zstd</application> command. The default is
> + Path to a <application>Zstd</application> command. The default is
> <literal>zstd</literal>, which will search for a command by that
> name in the configured <envar>PATH</envar>.
> </para></listitem>
Maybe it should say s/a/the/, like:
- Path to a <application>zstd</application> command. The default is
+ Path to the <application>zstd</application> command. The default is
^ permalink raw reply [nested|flat] 7+ messages in thread
* Re: typos
@ 2022-04-21 04:36 Michael Paquier <[email protected]>
parent: Alvaro Herrera <[email protected]>
1 sibling, 1 reply; 7+ messages in thread
From: Michael Paquier @ 2022-04-21 04:36 UTC (permalink / raw)
To: Alvaro Herrera <[email protected]>; +Cc: Justin Pryzby <[email protected]>; David Rowley <[email protected]>; pgsql-hackers; Fabien COELHO <[email protected]>; Amit Kapila <[email protected]>
On Wed, Apr 20, 2022 at 11:32:08PM +0200, Alvaro Herrera wrote:
> So the attached.
>
> --- a/doc/src/sgml/install-windows.sgml
> +++ b/doc/src/sgml/install-windows.sgml
> @@ -307,9 +307,9 @@ $ENV{MSBFLAGS}="/m";
> </varlistentry>
>
> <varlistentry>
> - <term><productname>ZSTD</productname></term>
> + <term><productname>Zstd</productname></term>
> <listitem><para>
> - Required for supporting <productname>ZSTD</productname> compression
> + Required for supporting <productname>Zstd</productname> compression
Looking at the zstd project itself for reference or just wiki-sensei,
I don't think that this is correct:
https://github.com/facebook/zstd
https://en.wikipedia.org/wiki/Zstd
Their README uses "zstd" in lower-case, while "Zstd" (first letter
upper-case) is used at the beginning of a sentence.
--
Michael
Attachments:
[application/pgp-signature] signature.asc (833B, ../../[email protected]/2-signature.asc)
download
^ permalink raw reply [nested|flat] 7+ messages in thread
* Re: typos
@ 2022-04-21 13:59 Peter Eisentraut <[email protected]>
parent: Michael Paquier <[email protected]>
0 siblings, 0 replies; 7+ messages in thread
From: Peter Eisentraut @ 2022-04-21 13:59 UTC (permalink / raw)
To: Michael Paquier <[email protected]>; Alvaro Herrera <[email protected]>; +Cc: Justin Pryzby <[email protected]>; David Rowley <[email protected]>; pgsql-hackers; Fabien COELHO <[email protected]>; Amit Kapila <[email protected]>
On 21.04.22 06:36, Michael Paquier wrote:
> On Wed, Apr 20, 2022 at 11:32:08PM +0200, Alvaro Herrera wrote:
>> So the attached.
>>
>> --- a/doc/src/sgml/install-windows.sgml
>> +++ b/doc/src/sgml/install-windows.sgml
>> @@ -307,9 +307,9 @@ $ENV{MSBFLAGS}="/m";
>> </varlistentry>
>>
>> <varlistentry>
>> - <term><productname>ZSTD</productname></term>
>> + <term><productname>Zstd</productname></term>
>> <listitem><para>
>> - Required for supporting <productname>ZSTD</productname> compression
>> + Required for supporting <productname>Zstd</productname> compression
>
> Looking at the zstd project itself for reference or just wiki-sensei,
> I don't think that this is correct:
> https://github.com/facebook/zstd
> https://en.wikipedia.org/wiki/Zstd
>
> Their README uses "zstd" in lower-case, while "Zstd" (first letter
> upper-case) is used at the beginning of a sentence.
It is referred to as "Zstandard" at both of those places. Maybe we
should use that. That is also easier to pronounce.
^ permalink raw reply [nested|flat] 7+ messages in thread
* Re: typos
@ 2022-05-11 02:03 Justin Pryzby <[email protected]>
parent: Justin Pryzby <[email protected]>
0 siblings, 1 reply; 7+ messages in thread
From: Justin Pryzby @ 2022-05-11 02:03 UTC (permalink / raw)
To: pgsql-hackers
I found a bunch more typos; a couple from codespell, and several which are the
result of looking for previously-reported typos, like:
time git log origin --grep '[tT]ypo' --word-diff -U1 |grep -Eo '\[-[[:lower:]]+-\]' |sed 's/^\[-//; s/-\]$//' |sort -u |grep -Fxvwf /usr/share/dict/words >badwords.txt
time grep -rhoFwf badwords.txt doc |sort -u >not-badwords.txt
time grep -Fxvwf not-badwords.txt ./badwords.txt >./badwords.txt.new
time grep -rhoIFwf ./badwords.txt.new src --incl='*.[chly]' --incl='*.p[lm]' |sort |uniq -c |sort -nr |less
^ permalink raw reply [nested|flat] 7+ messages in thread
* Re: typos
@ 2022-05-11 06:41 Michael Paquier <[email protected]>
parent: Justin Pryzby <[email protected]>
0 siblings, 0 replies; 7+ messages in thread
From: Michael Paquier @ 2022-05-11 06:41 UTC (permalink / raw)
To: Justin Pryzby <[email protected]>; +Cc: pgsql-hackers
On Tue, May 10, 2022 at 09:03:34PM -0500, Justin Pryzby wrote:
> I found a bunch more typos; a couple from codespell, and several which are the
> result of looking for previously-reported typos, like:
Thanks, applied 0002.
Regarding 0001, I don't really know which one of {AND,OR}ed or
{AND,OR}-ed is better. Note that the code prefers the former, but
your patch changes the docs to use the latter.
--
Michael
Attachments:
[application/pgp-signature] signature.asc (833B, ../../[email protected]/2-signature.asc)
download
^ permalink raw reply [nested|flat] 7+ messages in thread
end of thread, other threads:[~2022-05-11 06:41 UTC | newest]
Thread overview: 7+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2020-04-06 22:16 [PATCH v6 05/10] doc: s/evade/avoid/ Justin Pryzby <[email protected]>
2022-04-20 21:32 Re: typos Alvaro Herrera <[email protected]>
2022-04-20 21:38 ` Re: typos Justin Pryzby <[email protected]>
2022-05-11 02:03 ` Re: typos Justin Pryzby <[email protected]>
2022-05-11 06:41 ` Re: typos Michael Paquier <[email protected]>
2022-04-21 04:36 ` Re: typos Michael Paquier <[email protected]>
2022-04-21 13:59 ` Re: typos Peter Eisentraut <[email protected]>
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox