Fix typos

Started by Sehrope Sarkuniabout 7 years ago7 messageshackers
Beta feature

Hackorum builds and tests every patch posted to the lists, not only commitfest submissions. This is Hackorum's own CI rather than the PostgreSQL project's, and it is still under testing - please report anything that looks wrong.

won't retrysuccessCI history

You can run a PostgreSQL built from this patch straight from Docker, with no checkout and no build:

docker run --rm -p 5432:5432 ghcr.io/hackorum-dev/postgres-patch:t41010
psql -h localhost -U postgres

Built from patchset v6 (message #6), July 28, 2026 at 03:55 AM.

Every patchset is also pushed to a branch of our PostgreSQL fork, so you can check out the same tree CI built. Without a PostgreSQL checkout:

git clone --branch t41010_6 https://github.com/hackorum-dev/postgres.git

In a checkout you already have, add the fork once:

git remote add hackorum https://github.com/hackorum-dev/postgres.git

then, for this patchset and every later one:

git fetch hackorum t41010_6 && git checkout t41010_6

Patchset v6 (message #6) is on t41010_6

Jump to latest
#1Sehrope Sarkuni
sehrope@jackdb.com

Hi,

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

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

Attachments:

0001-Fix-typos.patchtext/x-patch; charset=US-ASCII; name=0001-Fix-typos.patchDownload+4-5
#2Michael Paquier
michael@paquier.xyz
In reply to: Sehrope Sarkuni (#1)
Re: Fix typos

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

#3Tom Lane
tgl@sss.pgh.pa.us
In reply to: Michael Paquier (#2)
Re: Fix typos

Michael Paquier <michael@paquier.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:

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

regards, tom lane

#4Alvaro Herrera
alvherre@2ndquadrant.com
In reply to: Tom Lane (#3)
Re: Fix typos

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:

/messages/by-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

#5Michael Paquier
michael@paquier.xyz
In reply to: Alvaro Herrera (#4)
Re: Fix typos

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

#6Sehrope Sarkuni
sehrope@jackdb.com
In reply to: Tom Lane (#3)
Re: Fix typos

On Thu, Aug 1, 2019 at 10:18 PM Tom Lane <tgl@sss.pgh.pa.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:

/messages/by-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/

Attachments:

t41010_6
v2-0001-Use-American-spelling-for-serialize-and-materalize.patchapplication/x-patch; name=v2-0001-Use-American-spelling-for-serialize-and-materalize.patchDownload+5-6
#7Sehrope Sarkuni
sehrope@jackdb.com
In reply to: Michael Paquier (#5)
Re: Fix typos

On Fri, Aug 2, 2019 at 12:11 AM Michael Paquier <michael@paquier.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/