Re: Fix typos

Lists: pgsql-hackers
From: Sehrope Sarkuni <sehrope(at)jackdb(dot)com>
To: Pg Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Fix typos
Date: 2019-08-01 12:24:17
Message-ID: CAH7T-arisDyLzcFFtjSt2JDPw-8ysvOXBiJW9mRU5CQ_=3DfCA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

Hi,

Attached fixes some typos for "serialise" => "serialize" and "materialise"
=> "materialize".

Regards,
-- Sehrope Sarkuni
Founder & CEO | JackDB, Inc. | https://www.jackdb.com/

Attachment Content-Type Size
0001-Fix-typos.patch text/x-patch 2.6 KB

From: Michael Paquier <michael(at)paquier(dot)xyz>
To: Sehrope Sarkuni <sehrope(at)jackdb(dot)com>
Cc: Pg Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Fix typos
Date: 2019-08-02 01:06:03
Message-ID: 20190802010603.GC1717@paquier.xyz
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

On Thu, Aug 01, 2019 at 08:24:17AM -0400, Sehrope Sarkuni wrote:
> Attached fixes some typos for "serialise" => "serialize" and "materialise"
> => "materialize".

These don't seem to be typos:
https://en.wiktionary.org/wiki/materialise
https://en.wiktionary.org/wiki/serialise
--
Michael


From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Michael Paquier <michael(at)paquier(dot)xyz>
Cc: Sehrope Sarkuni <sehrope(at)jackdb(dot)com>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Fix typos
Date: 2019-08-02 02:18:14
Message-ID: 838.1564712294@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

Michael Paquier <michael(at)paquier(dot)xyz> writes:
> On Thu, Aug 01, 2019 at 08:24:17AM -0400, Sehrope Sarkuni wrote:
>> Attached fixes some typos for "serialise" => "serialize" and "materialise"
>> => "materialize".

> These don't seem to be typos:
> https://en.wiktionary.org/wiki/materialise
> https://en.wiktionary.org/wiki/serialise

It's British vs. American spelling. For the most part, Postgres
follows American spelling, but there's the odd Briticism here and
there. I'm not sure whether it's worth trying to standardize.
I think the most recent opinion on this was Munro's:

/message-id/CA+hUKGJz-pdMgWXroiwvN-aeG4-AjdWj3gWdQKOSa8g65spdVw@mail.gmail.com

regards, tom lane


From: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Michael Paquier <michael(at)paquier(dot)xyz>, Sehrope Sarkuni <sehrope(at)jackdb(dot)com>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Fix typos
Date: 2019-08-02 03:01:59
Message-ID: 20190802030159.GA29394@alvherre.pgsql
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

On 2019-Aug-01, Tom Lane wrote:

> It's British vs. American spelling. For the most part, Postgres
> follows American spelling, but there's the odd Briticism here and
> there. I'm not sure whether it's worth trying to standardize.
> I think the most recent opinion on this was Munro's:
>
> /message-id/CA+hUKGJz-pdMgWXroiwvN-aeG4-AjdWj3gWdQKOSa8g65spdVw@mail.gmail.com

I think slight variations don't really detract from the value of the
product, and consider the odd variation a reminder of the diversity of
the project. I don't suggest that we purposefully introduce spelling
variations, or that we refrain from fixing ones that appear in code
we're changing, but I don't see the point in changing a line for the
sole reason of standardising the spelling of a word.

That said, I'm not a native English speaker.

--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services


From: Michael Paquier <michael(at)paquier(dot)xyz>
To: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Sehrope Sarkuni <sehrope(at)jackdb(dot)com>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Fix typos
Date: 2019-08-02 04:11:07
Message-ID: 20190802041107.GD19308@paquier.xyz
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

On Thu, Aug 01, 2019 at 11:01:59PM -0400, Alvaro Herrera wrote:
> I think slight variations don't really detract from the value of the
> product, and consider the odd variation a reminder of the diversity of
> the project. I don't suggest that we purposefully introduce spelling
> variations, or that we refrain from fixing ones that appear in code
> we're changing, but I don't see the point in changing a line for the
> sole reason of standardising the spelling of a word.

Agreed. This always reminds me of ANALYZE vs. ANALYSE where we don't
actually document the latter :)

> That said, I'm not a native English speaker.

Neither am I.
--
Michael


From: Sehrope Sarkuni <sehrope(at)jackdb(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Michael Paquier <michael(at)paquier(dot)xyz>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Fix typos
Date: 2019-08-02 10:37:44
Message-ID: CAH7T-apB97-5isHj=Bi+JSpaE9_L+pH7gPr=Mi-ZgR7bdpr7rQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

On Thu, Aug 1, 2019 at 10:18 PM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:

> It's British vs. American spelling. For the most part, Postgres
> follows American spelling, but there's the odd Briticism here and
> there.

Thanks for the explanation. I thought that might be the case but didn't
find any other usages of "serialise" so was not sure.

> I'm not sure whether it's worth trying to standardize.
> I think the most recent opinion on this was Munro's:
>
>
> /message-id/CA+hUKGJz-pdMgWXroiwvN-aeG4-AjdWj3gWdQKOSa8g65spdVw@mail.gmail.com

Either reads fine to me and the best rationale I can think of for going
with one spelling is to not have the same "fix" come up again.

If there is a desire to change this, attached is updated to include one
more instance of "materialise" and a change to the commit message to match
some similar ones I found in the past.

Regards,
-- Sehrope Sarkuni
Founder & CEO | JackDB, Inc. | https://www.jackdb.com/

Attachment Content-Type Size
v2-0001-Use-American-spelling-for-serialize-and-materalize.patch application/x-patch 3.3 KB

From: Sehrope Sarkuni <sehrope(at)jackdb(dot)com>
To: Michael Paquier <michael(at)paquier(dot)xyz>
Cc: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Fix typos
Date: 2019-08-02 10:45:08
Message-ID: CAH7T-aqYBBWCng++209tU9sF0jR4xqQGNkzS_ki6LMt5jeRkdA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

On Fri, Aug 2, 2019 at 12:11 AM Michael Paquier <michael(at)paquier(dot)xyz> wrote:

> On Thu, Aug 01, 2019 at 11:01:59PM -0400, Alvaro Herrera wrote:
> > I think slight variations don't really detract from the value of the
> > product, and consider the odd variation a reminder of the diversity of
> > the project. I don't suggest that we purposefully introduce spelling
> > variations, or that we refrain from fixing ones that appear in code
> > we're changing, but I don't see the point in changing a line for the
> > sole reason of standardising the spelling of a word.
>
> Agreed. This always reminds me of ANALYZE vs. ANALYSE where we don't
> actually document the latter :)
>

I didn't know about that. That's a fun one!

>
> > That said, I'm not a native English speaker.
>
> Neither am I.
>

I am. Consistency is nice but either reads fine to me. Only brought it up
as I didn't see many other usages so seemed out of place.

Regards,
-- Sehrope Sarkuni
Founder & CEO | JackDB, Inc. | https://www.jackdb.com/