Horton
Menu

Thread

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →

  1. Fix typo in planner.c

  2. Fix meson build of snowball code.

  3. Update to latest Snowball sources.

  4. Remove unneeded defines from pg_config.h.in

  5. Fix typos and inconsistencies in code and comments

  6. Fix a few more duplicate words in comments

  7. Fix a few duplicate words in comments

  8. Fix typos and grammar in the code

  9. Fix an assortment of spelling mistakes and typos

  10. Fix inconsistent LWLock tranche name "CommitTsSLRU"

  11. Fix typos in code comments and test data

  12. Fix typos and grammar in code comments and docs

  13. Fix a series of typos and outdated references

  14. Optimize InvalidateAttoptCacheCallback() and TypeCacheTypCallback()

  15. pg_upgrade: Remove unused macro

  16. pgcrypto: Remove unused binary from clean target

  17. Fix an assortment of typos

  18. Fix duplicated consecutive words in comments

  19. Remove unused function prototype

  20. Fix incorrect parameter name in prototype

  21. Fix typos and duplicate words

  22. Speedup 2PC recovery by skipping two phase state files in normal path

  23. Two-phase commit. Original patch by Heikki Linnakangas, with additional

  1. Typos in the code and README

    Daniel Gustafsson <daniel@yesql.se> — 2024-04-11T13:05:33Z

    Now that the tree has settled down a bit post-freeze I ran some tooling to
check spelling.  I was primarily interested in docs and README* which were
mostly free from simply typos, while the code had some in various comments and
one in code.  The attached fixes all that I came across (not cross-referenced
against ongoing reverts or any other fixup threads but will be before pushing
of course).

--
Daniel Gustafsson

  2. Re: Typos in the code and README

    Andrew Dunstan <andrew@dunslane.net> — 2024-04-11T13:21:06Z

    On 2024-04-11 Th 09:05, Daniel Gustafsson wrote:
> Now that the tree has settled down a bit post-freeze I ran some tooling to
> check spelling.  I was primarily interested in docs and README* which were
> mostly free from simply typos, while the code had some in various comments and
> one in code.  The attached fixes all that I came across (not cross-referenced
> against ongoing reverts or any other fixup threads but will be before pushing
> of course).



I have these covered:


src/test/modules/test_json_parser/README                  | 2 +-
.../test_json_parser/test_json_parser_incremental.c       | 4 ++--
src/test/modules/test_json_parser/test_json_parser_perf.c | 2 +-


cheers


andrew

--
Andrew Dunstan
EDB: https://www.enterprisedb.com

  3. Re: Typos in the code and README

    David Rowley <dgrowleyml@gmail.com> — 2024-04-11T13:29:58Z

    On Fri, 12 Apr 2024, 1:05 am Daniel Gustafsson, <daniel@yesql.se> wrote:

> Now that the tree has settled down a bit post-freeze I ran some tooling to
> check spelling.  I was primarily interested in docs and README* which were
> mostly free from simply typos, while the code had some in various comments
> and
> one in code.  The attached fixes all that I came across (not
> cross-referenced
> against ongoing reverts or any other fixup threads but will be before
> pushing
> of course).
>

I see you've corrected "iif" to "if". It should be "iff".

David

>

  4. Re: Typos in the code and README

    Daniel Gustafsson <daniel@yesql.se> — 2024-04-11T13:37:00Z

    > On 11 Apr 2024, at 15:29, David Rowley <dgrowleyml@gmail.com> wrote:
> 
> On Fri, 12 Apr 2024, 1:05 am Daniel Gustafsson, <daniel@yesql.se <mailto:daniel@yesql.se>> wrote:
>> Now that the tree has settled down a bit post-freeze I ran some tooling to
>> check spelling.  I was primarily interested in docs and README* which were
>> mostly free from simply typos, while the code had some in various comments and
>> one in code.  The attached fixes all that I came across (not cross-referenced
>> against ongoing reverts or any other fixup threads but will be before pushing
>> of course).
> 
> 
> I see you've corrected "iif" to "if". It should be "iff".

Gotcha, will fix.  I opted for "if" due to recent threads where the use of
"iff" was discouraged due to not being commonly known, but that was in doc/ and
not code comments.

--
Daniel Gustafsson

  5. Re: Typos in the code and README

    Bruce Momjian <bruce@momjian.us> — 2024-04-12T20:55:16Z

    On Thu, Apr 11, 2024 at 03:37:00PM +0200, Daniel Gustafsson wrote:
>     On 11 Apr 2024, at 15:29, David Rowley <dgrowleyml@gmail.com> wrote:
> 
>     On Fri, 12 Apr 2024, 1:05 am Daniel Gustafsson, <daniel@yesql.se> wrote:
> 
>         Now that the tree has settled down a bit post-freeze I ran some tooling
>         to
>         check spelling.  I was primarily interested in docs and README* which
>         were
>         mostly free from simply typos, while the code had some in various
>         comments and
>         one in code.  The attached fixes all that I came across (not
>         cross-referenced
>         against ongoing reverts or any other fixup threads but will be before
>         pushing
>         of course).
> 
> 
>     I see you've corrected "iif" to "if". It should be "iff".
> 
> 
> Gotcha, will fix.  I opted for "if" due to recent threads where the use of
> "iff" was discouraged due to not being commonly known, but that was in doc/ and
> not code comments.

I actually agree "iff" is just not clear enough.  "Iff" stands for "if
and only if" and maybe should be spelled out that way.

-- 
  Bruce Momjian  <bruce@momjian.us>        https://momjian.us
  EDB                                      https://enterprisedb.com

  Only you can decide what is important to you.

  6. Re: Typos in the code and README

    Bruce Momjian <bruce@momjian.us> — 2024-04-12T20:58:22Z

    On Fri, Apr 12, 2024 at 04:55:16PM -0400, Bruce Momjian wrote:
> On Thu, Apr 11, 2024 at 03:37:00PM +0200, Daniel Gustafsson wrote:
> >     On 11 Apr 2024, at 15:29, David Rowley <dgrowleyml@gmail.com> wrote:
> > 
> >     On Fri, 12 Apr 2024, 1:05 am Daniel Gustafsson, <daniel@yesql.se> wrote:
> > 
> >         Now that the tree has settled down a bit post-freeze I ran some tooling
> >         to
> >         check spelling.  I was primarily interested in docs and README* which
> >         were
> >         mostly free from simply typos, while the code had some in various
> >         comments and
> >         one in code.  The attached fixes all that I came across (not
> >         cross-referenced
> >         against ongoing reverts or any other fixup threads but will be before
> >         pushing
> >         of course).
> > 
> > 
> >     I see you've corrected "iif" to "if". It should be "iff".
> > 
> > 
> > Gotcha, will fix.  I opted for "if" due to recent threads where the use of
> > "iff" was discouraged due to not being commonly known, but that was in doc/ and
> > not code comments.
> 
> I actually agree "iff" is just not clear enough.  "Iff" stands for "if
> and only if" and maybe should be spelled out that way.

Just to clarify, I think "if and only if" means "if A then B" and B can
only happen if A happens, meaning there are not other cases where B can
happen.  This latter part is what disinguishes "iff" from "if".

-- 
  Bruce Momjian  <bruce@momjian.us>        https://momjian.us
  EDB                                      https://enterprisedb.com

  Only you can decide what is important to you.

  7. Re: Typos in the code and README

    Heikki Linnakangas <hlinnaka@iki.fi> — 2024-04-12T21:15:09Z

    On 11/04/2024 16:05, Daniel Gustafsson wrote:
> Now that the tree has settled down a bit post-freeze I ran some tooling to
> check spelling.  I was primarily interested in docs and README* which were
> mostly free from simply typos, while the code had some in various comments and
> one in code.  The attached fixes all that I came across (not cross-referenced
> against ongoing reverts or any other fixup threads but will be before pushing
> of course).

Here's a few more. I've accumulate these over the past couple of months, 
keeping them stashed in a branch, adding to it whenever I've spotted a 
minor typo while reading the code.

-- 
Heikki Linnakangas
Neon (https://neon.tech)

  8. Re: Typos in the code and README

    Daniel Gustafsson <daniel@yesql.se> — 2024-04-12T21:17:11Z

    > On 12 Apr 2024, at 23:15, Heikki Linnakangas <hlinnaka@iki.fi> wrote:
> 
> On 11/04/2024 16:05, Daniel Gustafsson wrote:
>> Now that the tree has settled down a bit post-freeze I ran some tooling to
>> check spelling.  I was primarily interested in docs and README* which were
>> mostly free from simply typos, while the code had some in various comments and
>> one in code.  The attached fixes all that I came across (not cross-referenced
>> against ongoing reverts or any other fixup threads but will be before pushing
>> of course).
> 
> Here's a few more. I've accumulate these over the past couple of months, keeping them stashed in a branch, adding to it whenever I've spotted a minor typo while reading the code.

Nice, let's lot all these together.

--
Daniel Gustafsson

  9. Re: Typos in the code and README

    David Rowley <dgrowleyml@gmail.com> — 2024-04-14T11:19:56Z

    On Sat, 13 Apr 2024 at 09:17, Daniel Gustafsson <daniel@yesql.se> wrote:
>
> > On 12 Apr 2024, at 23:15, Heikki Linnakangas <hlinnaka@iki.fi> wrote:
> > Here's a few more. I've accumulate these over the past couple of months, keeping them stashed in a branch, adding to it whenever I've spotted a minor typo while reading the code.
>
> Nice, let's lot all these together.

Here are a few additional ones to add to that.

Found with a manual trawl through git grep -E
'\b([a-zA-Z]{2,}[^long|^that])\s+\1\b' -- ':!*.po' ':!*.dat'

David

  10. Re: Typos in the code and README

    Daniel Gustafsson <daniel@yesql.se> — 2024-04-15T12:25:45Z

    > On 14 Apr 2024, at 13:19, David Rowley <dgrowleyml@gmail.com> wrote:
> 
> On Sat, 13 Apr 2024 at 09:17, Daniel Gustafsson <daniel@yesql.se> wrote:
>> 
>>> On 12 Apr 2024, at 23:15, Heikki Linnakangas <hlinnaka@iki.fi> wrote:
>>> Here's a few more. I've accumulate these over the past couple of months, keeping them stashed in a branch, adding to it whenever I've spotted a minor typo while reading the code.
>> 
>> Nice, let's lot all these together.
> 
> Here are a few additional ones to add to that.

Thanks.  Collecting all the ones submitted here, as well as a few submitted
off-list by Alexander, the patch is now a 3-part patchset of cleanups:

0001 contains the typos and duplicate words fixups, 0002 fixes a parameter with
the wrong name in the prototype and 0003 removes a leftover prototype which was
accidentally left in a refactoring.

--
Daniel Gustafsson

  11. Re: Typos in the code and README

    Richard Guo <guofenglinux@gmail.com> — 2024-04-16T08:28:24Z

    On Mon, Apr 15, 2024 at 8:26 PM Daniel Gustafsson <daniel@yesql.se> wrote:

> Thanks.  Collecting all the ones submitted here, as well as a few submitted
> off-list by Alexander, the patch is now a 3-part patchset of cleanups:
>
> 0001 contains the typos and duplicate words fixups, 0002 fixes a parameter
> with
> the wrong name in the prototype and 0003 removes a leftover prototype
> which was
> accidentally left in a refactoring.


BTW, it seems that 0001 needs a rebase over 9dfcac8e15.

Thanks
Richard

  12. Re: Typos in the code and README

    Nazir Bilal Yavuz <byavuz81@gmail.com> — 2024-04-16T13:37:50Z

    Hi,

Thanks for working on this!

On Mon, 15 Apr 2024 at 15:26, Daniel Gustafsson <daniel@yesql.se> wrote:
>
> > On 14 Apr 2024, at 13:19, David Rowley <dgrowleyml@gmail.com> wrote:
> >
> > On Sat, 13 Apr 2024 at 09:17, Daniel Gustafsson <daniel@yesql.se> wrote:
> >>
> >>> On 12 Apr 2024, at 23:15, Heikki Linnakangas <hlinnaka@iki.fi> wrote:
> >>> Here's a few more. I've accumulate these over the past couple of months, keeping them stashed in a branch, adding to it whenever I've spotted a minor typo while reading the code.
> >>
> >> Nice, let's lot all these together.
> >
> > Here are a few additional ones to add to that.
>
> Thanks.  Collecting all the ones submitted here, as well as a few submitted
> off-list by Alexander, the patch is now a 3-part patchset of cleanups:
>
> 0001 contains the typos and duplicate words fixups, 0002 fixes a parameter with
> the wrong name in the prototype and 0003 removes a leftover prototype which was
> accidentally left in a refactoring.

I realized two small typos: 'sgmr' -> 'smgr'. You may want to include
them in 0001.

-- 
Regards,
Nazir Bilal Yavuz
Microsoft

  13. Re: Typos in the code and README

    Daniel Gustafsson <daniel@yesql.se> — 2024-04-19T08:13:13Z

    > On 16 Apr 2024, at 15:37, Nazir Bilal Yavuz <byavuz81@gmail.com> wrote:

> I realized two small typos: 'sgmr' -> 'smgr'. You may want to include
> them in 0001.

Thanks, I incorporated these into 0001 before pushing.  All the commits in this
patchset are now applied.

--
Daniel Gustafsson

  14. Re: Typos in the code and README

    David Rowley <dgrowleyml@gmail.com> — 2024-04-20T04:09:13Z

    On Fri, 19 Apr 2024 at 20:13, Daniel Gustafsson <daniel@yesql.se> wrote:
> Thanks, I incorporated these into 0001 before pushing.  All the commits in this
> patchset are now applied.

Here are a few more to see if it motivates anyone else to do a more
thorough search for another batch.

Fixes duplicate words spanning multiple lines plus an outdated
reference to "streaming read" which was renamed to "read stream" late
in that patch's development.

duplicate words found using:
ag "\s([a-zA-Z]{2,})[\s*]*\n\1\b"
ag "\s([a-zA-Z]{2,})\n(\s*\*\s*)\1\b"

David

  15. Re: Typos in the code and README

    David Rowley <dgrowleyml@gmail.com> — 2024-04-28T08:05:06Z

    On Sat, 20 Apr 2024 at 16:09, David Rowley <dgrowleyml@gmail.com> wrote:
> Here are a few more to see if it motivates anyone else to do a more
> thorough search for another batch.

I've pushed these now.

David

  16. Re: Typos in the code and README

    Alexander Lakhin <exclusion@gmail.com> — 2024-05-02T12:00:00Z

    Hello,

28.04.2024 11:05, David Rowley wrote:

> On Sat, 20 Apr 2024 at 16:09, David Rowley <dgrowleyml@gmail.com> wrote:
>> Here are a few more to see if it motivates anyone else to do a more
>> thorough search for another batch.
> I've pushed these now.

Please look also at the list of other typos and inconsistencies I've found
on the master branch:
additional_notnulls -> old_notnulls (cf. b0e96f311)
ATAddCheckConstraint ->? ATAddCheckNNConstraint (cf. b0e96f311)
bt_page_check -> bt_target_page_check (cf. 5ae208720)
calblack_arg -> callback_arg
combinig -> combining
compaining -> complaining
ctllock - remove (cf. 53c2a97a9)
dabatase -> database
eval_const_exprs_mutator -> eval_const_expressions_mutator
ExecEndValuesScan - remove (cf. d060e921e)
get_json_nested_columns -> get_json_table_nested_columns
io_direct -> debug_io_direct
iS ->? IS (a neatnik-ism)
joing ->? join (cf. 20f90a0e4)
_jumbleRangeTblEntry - remove (cf. 367c989cd)
Mallroy -> Mallory
onstead -> instead
procedual -> procedural
psql_safe -> safe_psql
read_quoted_pattern -> read_quoted_string
repertiore -> repertoire
rightfirstdataoffset ->? rightfirstoffset (cf. 5ae208720)
Sincle ->? Since (perhaps the whole sentence could be rephrased)
sslnegotition=direct/requiredirectwould -> sslnegotiation=direct/requiredirect would
sslnegotitation -> sslnegotiation
walreciever -> walreceiver
xid_wrapround -> xid_wraparound

(some of them are located in doc/, so it's not a code-only change)
I've attached the patch for your convenience, though maybe some
of the suggestions are to be discarded.

Best regards,
Alexander

  17. Re: Typos in the code and README

    David Rowley <dgrowleyml@gmail.com> — 2024-05-03T14:36:35Z

    On Fri, 3 May 2024 at 00:00, Alexander Lakhin <exclusion@gmail.com> wrote:
> (some of them are located in doc/, so it's not a code-only change)
> I've attached the patch for your convenience, though maybe some
> of the suggestions are to be discarded.

Thanks. I was hoping you'd do that.

I pushed the patch after only adjusting the path in the docs which had
"module" rather than "modules".

David

  18. Re: Typos in the code and README

    Alexander Lakhin <exclusion@gmail.com> — 2024-07-04T17:00:01Z

    Hello hackers,

03.05.2024 17:36, David Rowley wrote:
> I pushed the patch after only adjusting the path in the docs which had
> "module" rather than "modules".

Please look at another bunch of inconsistencies/orphaned entities I found
in the tree, with the possible substitutions:
errmsg_buf -> errormsg_buf
(coined by 6b18b3fe2)

NoMovementScanDirectionScans -> NoMovementScanDirection
(introduced with e9aaf0632, discussed in [1], but still seems inaccurate)

XLogReadRecordInternal -> XLogReadRecord
(from 3f1ce9734, align with a comment above: "Start and end point of last
record returned by XLogReadRecord().")

BYPASS_ALLOWCONN -> BGWORKER_BYPASS_ROLELOGINCHECK (see 492217301)

xs_ctup.t_self -> xs_heaptid (see c2fe139c2 and 304532421)

pgStatShmLookupCache -> pgStatLocal.shmem (coined by 5891c7a8e)

smgr_fsm_nblocks and smgr_vm_nblocks -> smgr_cached_nblocks
(see the same comment updated by c5315f4f4)

XID becomes older than GlobalXmin -> XID becomes visible to everyone
(in accordance with dc7420c2c9 src/backend/access/gist/gistutil.c)

gen-rtab - remove (non-existing since db7d1a7b0)

BARRIER_SHOULD_CHECK - remove (unused since a3ed4d1ef)

EXE_EXT - remove (unused since f06b1c598)

endterm - remove
(see 60c90c16c -- Use xreflabel attributes instead of endterm ...)

xl_commit_ts_set, SizeOfCommitTsSet - remove (unused since 08aa89b32)

The corresponding patch is attached for your convenience.

[1] https://www.postgresql.org/message-id/20230131140224.7j6gbcsfwmad2a4b%40liskov

Best regards,
Alexander

  19. Re: Typos in the code and README

    Daniel Gustafsson <daniel@yesql.se> — 2024-07-25T13:45:32Z

    > On 4 Jul 2024, at 19:00, Alexander Lakhin <exclusion@gmail.com> wrote:

> Please look at another bunch of inconsistencies/orphaned entities I found
> in the tree, with the possible substitutions:

Thanks for these, and sorry for the delay in processing them (summer happened
etc).  I've started to go through them and have applied some low-hanging fruit
already, will go over all them.

--
Daniel Gustafsson

  20. Re: Typos in the code and README

    David Rowley <dgrowleyml@gmail.com> — 2024-08-12T11:59:31Z

    (I know Daniel mentioned he'd get to these, but the ScanDirection one
was my fault and I needed to clear that off my mind. I did a few
others while on this topic.)

On Fri, 5 Jul 2024 at 05:00, Alexander Lakhin <exclusion@gmail.com> wrote:
> Please look at another bunch of inconsistencies/orphaned entities I found
> in the tree, with the possible substitutions:
> errmsg_buf -> errormsg_buf
> (coined by 6b18b3fe2)

Fixed.

> NoMovementScanDirectionScans -> NoMovementScanDirection
> (introduced with e9aaf0632, discussed in [1], but still seems inaccurate)

Oops. Fixed.

> XLogReadRecordInternal -> XLogReadRecord
> (from 3f1ce9734, align with a comment above: "Start and end point of last
> record returned by XLogReadRecord().")

Fixed.

> BYPASS_ALLOWCONN -> BGWORKER_BYPASS_ROLELOGINCHECK (see 492217301)

Fixed

> xs_ctup.t_self -> xs_heaptid (see c2fe139c2 and 304532421)

Fixed.

> pgStatShmLookupCache -> pgStatLocal.shmem (coined by 5891c7a8e)

Fixed.

> smgr_fsm_nblocks and smgr_vm_nblocks -> smgr_cached_nblocks
> (see the same comment updated by c5315f4f4)

Heikki fixed in 19de089cd.

> XID becomes older than GlobalXmin -> XID becomes visible to everyone
> (in accordance with dc7420c2c9 src/backend/access/gist/gistutil.c)

I'd need to spend more time to understand this.

> gen-rtab - remove (non-existing since db7d1a7b0)

Daniel fixed in cc59f9d0f.

> BARRIER_SHOULD_CHECK - remove (unused since a3ed4d1ef)

I wasn't sure if nothing else external could be using this. The macro
doesn't use any fields that are now gone, so I'm not confident enough
to remove it. Can Robert confirm?

> EXE_EXT - remove (unused since f06b1c598)

Daniel fixed in 88e3da565.

> endterm - remove
> (see 60c90c16c -- Use xreflabel attributes instead of endterm ...)

I read that commit message and I agree it's now unused. I just didn't
get any vibes from the commit message that it shouldn't ever be used
again.  Can Tom confirm?

> xl_commit_ts_set, SizeOfCommitTsSet - remove (unused since 08aa89b32)

I would say it should be removed.  I just didn't because the commit I
had pending fitted into the "typos" category and this didn't quite
fit.

I've attached a patch with the remainder.

David

  21. Re: Typos in the code and README

    Alexander Lakhin <exclusion@gmail.com> — 2024-09-02T18:00:00Z

    Hello,

12.08.2024 14:59, David Rowley wrote:
> (I know Daniel mentioned he'd get to these, but the ScanDirection one
> was my fault and I needed to clear that off my mind. I did a few
> others while on this topic.)

Thank you, David, for working on that!

I've gathered another bunch of defects with the possible substitutions.
Please take a look:
adapated -> adapted

becasue -> because

cancelled -> canceled (introduced by 90f517821, but see 8c9da1441)

cange -> change

comand -> command

CommitTSSLRU -> CommitTsSLRU (introduced by 53c2a97a9; maybe the fix
  should be back-patched...)

connectOptions2 -> pqConnectOptions2 (see 774bcffe4)

Injections points -> Injection points

jsetate -> jsestate

LockShmemSize -> remove the sentence? (added by ec0baf949, outdated with
  a794fb068)

MaybeStartSlotSyncWorker -> LaunchMissingBackgroundProcesses (the logic to
  start B_SLOTSYNC_WORKER moved from the former to the latter function with
  3354f8528)

multixact_member_buffer -> multixact_member_buffers

per_data_data -> per_buffer_data (see code below the comment; introduced by
  b5a9b18cd)

per_buffer_private -> remove the function declaration? (the duplicate
  declaration was added by a858be17c)

performancewise -> performance-wise? (coined by a7f107df2)

pgstat_add_kind -> pgstat_register_kind (see 7949d9594)

pg_signal_autovacuum -> pg_signal_autovacuum_worker (see d2b74882c)

recoveery -> recovery

RegisteredWorker -> RegisteredBgWorker

RUNNING_XACT -> RUNNING_XACTS

sanpshot -> snapshot

TypeEntry -> TypeCacheEntry (align with AttoptCacheEntry, from the same
  commit 40064a8ee)

The corresponding patch is attached for your convenience.

Best regards,
Alexander

  22. Re: Typos in the code and README

    Michael Paquier <michael@paquier.xyz> — 2024-09-03T05:24:32Z

    On Mon, Sep 02, 2024 at 09:00:00PM +0300, Alexander Lakhin wrote:
> I've gathered another bunch of defects with the possible substitutions.
> Please take a look:
> pgstat_add_kind -> pgstat_register_kind (see 7949d9594)

And here I thought I took care of these inconsistencies.  This one is
on me so I'll go fix that in a bit, with the rest while on it.
--
Michael

  23. Re: Typos in the code and README

    Michael Paquier <michael@paquier.xyz> — 2024-09-03T05:51:40Z

    On Tue, Sep 03, 2024 at 02:24:32PM +0900, Michael Paquier wrote:
> On Mon, Sep 02, 2024 at 09:00:00PM +0300, Alexander Lakhin wrote:
> > I've gathered another bunch of defects with the possible substitutions.
> > Please take a look:
> > pgstat_add_kind -> pgstat_register_kind (see 7949d9594)
> 
> And here I thought I took care of these inconsistencies.  This one is
> on me so I'll go fix that in a bit, with the rest while on it.

And done that.

The bit about CommitTSSLRU -> CommitTsSLRU in lwlock.c should be
backpatched down to 17, indeed, but the branch is frozen until the RC
tag lands in the tree, so I have left it out for now. The tag should
show up tomorrow or so.  Good thing that you have noticed this issue
before the release.
--
Michael

  24. Re: Typos in the code and README

    Daniel Gustafsson <daniel@yesql.se> — 2024-09-03T10:00:13Z

    > On 3 Sep 2024, at 07:51, Michael Paquier <michael@paquier.xyz> wrote:
> 
> On Tue, Sep 03, 2024 at 02:24:32PM +0900, Michael Paquier wrote:
>> On Mon, Sep 02, 2024 at 09:00:00PM +0300, Alexander Lakhin wrote:
>>> I've gathered another bunch of defects with the possible substitutions.
>>> Please take a look:
>>> pgstat_add_kind -> pgstat_register_kind (see 7949d9594)
>> 
>> And here I thought I took care of these inconsistencies.  This one is
>> on me so I'll go fix that in a bit, with the rest while on it.
> 
> And done that.
> 
> The bit about CommitTSSLRU -> CommitTsSLRU in lwlock.c should be
> backpatched down to 17, indeed, but the branch is frozen until the RC
> tag lands in the tree, so I have left it out for now. The tag should
> show up tomorrow or so.  Good thing that you have noticed this issue
> before the release.

I see your v17 typo fixes, and raise you a few more.  Commit 31a98934d169 from
just now contains 2 (out of 3) sets of typos introduced in v17 so they should
follow along when you push the ones mentioned here.

--
Daniel Gustafsson

  25. Re: Typos in the code and README

    Michael Paquier <michael@paquier.xyz> — 2024-09-04T01:25:17Z

    On Tue, Sep 03, 2024 at 12:00:13PM +0200, Daniel Gustafsson wrote:
> I see your v17 typo fixes, and raise you a few more.  Commit 31a98934d169 from
> just now contains 2 (out of 3) sets of typos introduced in v17 so they should
> follow along when you push the ones mentioned here.

Is that really mandatory?  I tend to worry about back branches only
when this stuff is user-visible, like in the docs or error messages.
This opinion varies for each individual, of course.  That's just my
lazy opinion.

CommitTSSLRU -> CommitTsSLRU is user-visible, showing up in
pg_stat_activity.  Fixed this one with 08b9b9e043bb, as the tag for
17rc1 has been pushed.

Picking f747bc18f7f2 and 75c5231a00f3 on REL_17_STABLE leads to the
attached, I think, without the conflicts.
--
Michael

  26. Re: Typos in the code and README

    Daniel Gustafsson <daniel@yesql.se> — 2024-09-04T08:23:44Z

    > On 4 Sep 2024, at 03:25, Michael Paquier <michael@paquier.xyz> wrote:
> 
> On Tue, Sep 03, 2024 at 12:00:13PM +0200, Daniel Gustafsson wrote:
>> I see your v17 typo fixes, and raise you a few more.  Commit 31a98934d169 from
>> just now contains 2 (out of 3) sets of typos introduced in v17 so they should
>> follow along when you push the ones mentioned here.
> 
> Is that really mandatory?  I tend to worry about back branches only
> when this stuff is user-visible, like in the docs or error messages.
> This opinion varies for each individual, of course.  That's just my
> lazy opinion.

Not mandatory at all, but since you were prepping a typo backpatch anyways I
figured these could join to put a small dent in reducing risks for future
backports.

--
Daniel Gustafsson

  27. Re: Typos in the code and README

    David Rowley <dgrowleyml@gmail.com> — 2024-09-04T15:34:31Z

    On Wed, 4 Sept 2024 at 20:24, Daniel Gustafsson <daniel@yesql.se> wrote:
> Not mandatory at all, but since you were prepping a typo backpatch anyways I
> figured these could join to put a small dent in reducing risks for future
> backports.

I think this is pretty good logic.  I think fixing comment typos in
ancient code and backpatching to all supported versions isn't good use
of time, but fixing a typo in "recent" code and backpatching to where
that code was added seems useful. Newer code is more likely to need
bug fixes in the future, so going to a bit more effort to make
backpatching those bug fixes easier seems worth the effort.  I just
don't know what "recent" should be defined as. I'd say if it's in a
version we've not released yet, that's probably recent. By the time .1
is out, there's less chance of bugs in new code. Anyway, I doubt hard
guidelines are warranted here, but maybe some hints about best
practices in https://wiki.postgresql.org/wiki/Committing_checklist ?

David

  28. Re: Typos in the code and README

    Michael Paquier <michael@paquier.xyz> — 2024-09-05T00:03:26Z

    On Thu, Sep 05, 2024 at 03:34:31AM +1200, David Rowley wrote:
> Anyway, I doubt hard
> guidelines are warranted here, but maybe some hints about best
> practices in https://wiki.postgresql.org/wiki/Committing_checklist ?

Yep, that may be useful.  I just tend to be cautious because it can be
very easy to mess up things depending on the code path you're
manipulating, speaking with some..  Experience on the matter.  And an
RC1 is kind of something to be cautious with.
--
Michael

  29. Re: Typos in the code and README

    Daniel Gustafsson <daniel@yesql.se> — 2024-09-06T09:42:20Z

    > On 4 Sep 2024, at 17:34, David Rowley <dgrowleyml@gmail.com> wrote:
> 
> On Wed, 4 Sept 2024 at 20:24, Daniel Gustafsson <daniel@yesql.se> wrote:
>> Not mandatory at all, but since you were prepping a typo backpatch anyways I
>> figured these could join to put a small dent in reducing risks for future
>> backports.
> 
> I think this is pretty good logic.  I think fixing comment typos in
> ancient code and backpatching to all supported versions isn't good use
> of time, but fixing a typo in "recent" code and backpatching to where
> that code was added seems useful. Newer code is more likely to need
> bug fixes in the future, so going to a bit more effort to make
> backpatching those bug fixes easier seems worth the effort.

Absolutely agree.

> I just don't know what "recent" should be defined as. I'd say if it's in a
> version we've not released yet, that's probably recent. By the time .1
> is out, there's less chance of bugs in new code. Anyway, I doubt hard
> guidelines are warranted here, but maybe some hints about best
> practices in https://wiki.postgresql.org/wiki/Committing_checklist ?

That sounds like a good idea.  Off the cuff I would agree that unreleased
versions and .0 versions are strong candidates (but not mandatory) for trivial
backpatches like typos, beyond that the value is likely to be lower.

--
Daniel Gustafsson

  30. Re: Typos in the code and README

    Alexander Lakhin <exclusion@gmail.com> — 2025-01-01T16:00:01Z

    Hello hackers,

03.09.2024 08:51, Michael Paquier wrote:
> And done that.

Please look at another collection of typos and inconsistencies introduced
during 2024:
behvior -> behavior

contraint -> constraint

curent -> current

disable_node -> disabled_nodes

disabled_node > disabled_nodes

disable_nodes -> disabled_nodes

FindFkPeriodOpers -> FindFKPeriodOpers

heap_inplace_update -> heap_inplace_update_and_unlock

InitLocks -> LockManagerShmemInit? (see fbce7dfc7)

isReadOnly -> IsReadOnly

negotations -> negotiations

notnull_inh -> notnull_islocal (coined by 14e87ffa5)

num_word -> just remove an old-fashioned comment? or change to
  append_num_word (see 5bf948d56)

objectAddress -> ObjectAddress

orderProc -> orderProcs

pg_visibilitymap.c -> visibilitymap.c

postitional_fcinfo -> positional_fcinfo

preserve_formatting -> preserve_comments (see 995e0fbc1)

procesess -> processes

trapper -> trapped

trasformed -> transformed

vac_update_datfrozenid -> vac_update_datfrozenxid

XLPH_HAS_* -> XLHP_HAS_*

XLogRecordInsert -> XLogInsertRecord (coined by afd12774a)

Please find attached the corresponding patch for your convenience.

Best regards,
Alexander

  31. Re: Typos in the code and README

    David Rowley <dgrowleyml@gmail.com> — 2025-01-01T23:22:35Z

    On Thu, 2 Jan 2025 at 05:00, Alexander Lakhin <exclusion@gmail.com> wrote:
> Please look at another collection of typos and inconsistencies introduced
> during 2024:

The fixes all look good to me. I see some are mine, so I will take
care of pushing.

Thanks for the patch.

David

  32. Re: Typos in the code and README

    Alexander Lakhin <exclusion@gmail.com> — 2025-04-19T08:00:00Z

    Hello hackers,

I've gathered the following collection of typos and inconsistencies
appeared since 2025-01-01:
aio_worker.c -> method_worker.c

amcheck_lock_relation -> amcheck_lock_relation_and_check

be_key -> be_cancel_key

belonds -> belongs

cann -> can

CheckConstraintFetch -> CheckNNConstraintFetch
(renamed by a379061a2)

COLLID -> COLLOID

coltypemods -> coltypmods

columsn -> column

combinig -> combining

containting -> containing

non-exist datbase -> non-existent database

dead_end -> dead-end
(see a78af0427)

derefefence -> dereference

displyed -> displayed

droping -> dropping

es_eqp_active -> es_epq_active

es_part_prune_result -> es_part_prune_results

ExceDelete -> ExecDelete

excludes-chema -> exclude-schema

it fhe -> if the

GIN_TUPLE_ -> GIN_TUPLE_H

HandleFatalErrors -> HandleFatalError

happend -> reached
(to preserve the line width)

happenning -> happening

HIKEY -> P_HIKEY

IGNORE_CHECKSUM_FAILURE -> PIV_IGNORE_CHECKSUM_FAILURE

InitExecPartitionPruneContext -> InitExecPartitionPruneContexts

inititalization -> initialization

Injection_point -> Injection point

iterm -> item

$log_end -> remove unused variable
(align with src/test/modules/oauth_validator/t/001_server.pl)

looplback -> loopback

mappping -> mapping

MyWorkerId -> MyIoWorkerId

negotation -> negotiation

network-byte-order -> network byte order

parititioned* -> partitioned*

pending_null -> pending_nulls

permanentaly -> permanently

pg_localeconf_free -> pg_localeconv_free

PG_READ_ALL_STATS -> ROLE_PG_READ_ALL_STATS

pk_strategy -> pk_cmptype
(see 8123e91f5)

PM_WAIT_BACKEND -> PM_WAIT_BACKENDS

pollset -> poll set

predicable -> predictable

PRINT_STATS_STDERR -> PRINT_STATS_TO_STDERR

procced -> proceed

remining -> remaining

save_xmin -> saved_xmin

smgstartreadv -> smgrstartreadv

StartLocalBufer -> StartLocalBufferIO

statitistics -> statistics

swaping -> swapping

synq_queue_destroy -> sync_queue_destroy

thats -> that's

ther -> there

test-server -> test server

unfronzen -> unfrozen

upin -> unpin

Please find attached the complete patch for your convenience.

Best regards,
Alexander Lakhin
Neon (https://neon.tech)

  33. Re: Typos in the code and README

    Michael Paquier <michael@paquier.xyz> — 2025-04-19T10:04:28Z

    On Sat, Apr 19, 2025 at 11:00:00AM +0300, Alexander Lakhin wrote:
> I've gathered the following collection of typos and inconsistencies
> appeared since 2025-01-01:

Good finds, Alexander, particularly for the set of function and
variable names used in the comments that do not match with the
reality.  I've checked all these, and they're good for me.

> GIN_TUPLE_ -> GIN_TUPLE_H

This one is interested.  It should be harmless in practice, but it
could cause conflicts in theory.
--
Michael

  34. Re: Typos in the code and README

    Alexander Lakhin <exclusion@gmail.com> — 2025-04-19T11:00:00Z

    Hello Michael,

19.04.2025 13:04, Michael Paquier wrote:
> Good finds, Alexander, particularly for the set of function and
> variable names used in the comments that do not match with the
> reality.  I've checked all these, and they're good for me.

Thank you for your attention to this!

>> GIN_TUPLE_ -> GIN_TUPLE_H
> This one is interested.  It should be harmless in practice, but it
> could cause conflicts in theory.
> --

I've included it in the collection because of:
#ifndef GIN_TUPLE_
...
#endif                                                   /* GIN_TUPLE_H */

Best regards,
Alexander Lakhin
Neon (https://neon.tech)

  35. Re: Typos in the code and README

    David Rowley <dgrowleyml@gmail.com> — 2025-04-20T01:32:19Z

    On Sat, 19 Apr 2025 at 20:00, Alexander Lakhin <exclusion@gmail.com> wrote:
> I've gathered the following collection of typos and inconsistencies
> appeared since 2025-01-01:

Here are a few more fixes of a similar ilk. All new in 2025,
predominantly from the last few days before feature freeze.

The few I wasn't a full 100% certain on were "big big" *could* be used
as an adjective, but "huge" or "massive" would have been better. I
didn't see the need so assumed it was a mistake. The "now now"
technically is ok in South Africa [1], but I don't believe that
meaning was the intention.

I found these with a manual trawl through:

git grep -E "\b(\w+[^long|^that]+)\s+\1\b" -- ':!*.po' ':!*.out'
':!*.data' ':!*.dat' ':!*.xml'

David

[1] https://en.wikipedia.org/wiki/List_of_South_African_slang_words

  36. Re: Typos in the code and README

    David Rowley <dgrowleyml@gmail.com> — 2025-04-20T22:42:19Z

    On Sun, 20 Apr 2025 at 13:32, David Rowley <dgrowleyml@gmail.com> wrote:
> Here are a few more fixes of a similar ilk. All new in 2025,
> predominantly from the last few days before feature freeze.

I've pushed those ones.

David

  37. Re: Typos in the code and README

    Michael Paquier <michael@paquier.xyz> — 2025-04-20T23:21:12Z

    On Mon, Apr 21, 2025 at 10:42:19AM +1200, David Rowley wrote:
> On Sun, 20 Apr 2025 at 13:32, David Rowley <dgrowleyml@gmail.com> wrote:
>> Here are a few more fixes of a similar ilk. All new in 2025,
>> predominantly from the last few days before feature freeze.
> 
> I've pushed those ones.

Thanks for sharing the command able to spot all these.
--
Michael

  38. Re: Typos in the code and README

    David Rowley <dgrowleyml@gmail.com> — 2025-04-21T01:52:53Z

    On Mon, 21 Apr 2025 at 11:21, Michael Paquier <michael@paquier.xyz> wrote:
> Thanks for sharing the command able to spot all these.

I just pushed a few more. The previous regex didn't account for the
duplicate word being on the next line. I dug up the following to find
the ones just committed in 78eda9e26.

ag "\s([a-zA-Z]{2,})[\s*]*\n\1\b"
ag "\s([a-zA-Z]{2,})\n(\s*\*\s*)\1\b"

cd doc
ag "\s([a-zA-Z]{1,})[\s*]*\n\s+\1\b"

David

  39. Re: Typos in the code and README

    Alexander Lakhin <exclusion@gmail.com> — 2026-01-01T08:00:00Z

    Hello hackers,

Please look at another collection of typos, inconsistencies, and lost
entities (most of them introduced in 2025, but not all). In passing, I'm
proposing to synchronize parameter names in function declarations with
implementations.

Typos:
all-forzen -> all-frozen
Historcally -> Historically
insallation -> installation
maintanence_work_mem -> maintenance_work_mem
ndicates -> indicates
noacess -> noaccess
parititoned -> partititoned

Inconsistent spelling:
brin_wi_index -> brin_wi_idx
bufhdr -> bufHdr
FullTransactionID -> FullTransactionId
GroupExprInfos - > GroupingExprInfos
HEAP_PRUNE_FREEZE -> HEAP_PAGE_PRUNE_FREEZE (per 1937ed706)
inferPredExprs -> inferIndexExprs
newRightLink -> newRightlink
paramtypmode -> paramtypmod
pg_stat_replication_slot -> pg_stat_replication_slots
READ_WRITE_PARSE_PLAN_TREES -> WRITE_READ_PARSE_PLAN_TREES
DEFAULT_DEBUG_READ_WRITE_PARSE_PLAN_TREES -> DEFAULT_DEBUG_WRITE_READ_PARSE_PLAN_TREES
ReloptInfo -> RelOptInfo
retain_conflict_info -> retain_dead_tuples (see a850be2fe)
TIDStore -> TidStore

Inconsistent parameter naming:
RangeVarCallbackForStats(..., Oid oldRelid, ...) -> RangeVarCallbackForStats(..., Oid oldRelId, ...)
heap_lock_updated_tuple(..., TransactionId prior_rawxmax, ...) -> heap_lock_updated_tuple(..., TransactionId 
prior_raw_xmax, ...)
StartupLogicalDecodingStatus(bool status_in_control_file) -> StartupLogicalDecodingStatus(bool last_status)
FetchRelationStates(..., bool *has_pending_sequences, ...) -> FetchRelationStates(..., bool *has_pending_subsequences, ...)
PredicateLockTwoPhaseFinish(FullTransactionId xid, -> PredicateLockTwoPhaseFinish(FullTransactionId fxid,
PredicateLockTwoPhaseFinish(TransactionId xid, -> PredicateLockTwoPhaseFinish(FullTransactionId fxid,
SetSequence(Oid relid, int64 next, bool is_called) -> SetSequence(Oid relid, int64 next, bool iscalled)
fdatasync(int fildes) -> fdatasync(int fd)
gai_strerror(int ecode) -> gai_strerror(int errcode)
llvm_copy_attributes(LLVMValueRef from, LLVMValueRef to) -> llvm_copy_attributes(LLVMValueRef v_from, LLVMValueRef v_to)
ox1(PlannerInfo *root, Gene *mom, Gene *dad, ...) -> ox1(PlannerInfo *root, Gene *tour1, Gene *tour2, ...)
ox2(PlannerInfo *root, Gene *mom, Gene *dad, ...) -> ox2(PlannerInfo *root, Gene *tour1, Gene *tour2, ...)
pg_pread(..., size_t nbyte, ...) -> pg_pread(..., size_t size, ...)
pg_pwrite(..., size_t nbyte, ...) -> pg_pwrite(..., size_t size, ...)
pgwin32_connect(..., int namelen) -> pgwin32_connect(..., int addrlen)
replace_s(..., int * adjustment) -> replace_s(..., int * adjptr)

There is also
extern int        pgwin32_recv(SOCKET s, char *buf, int len, int flags);
vs
pgwin32_recv(SOCKET s, char *buf, int len, int f)
but I've left at as-is; probably the change should be made in the
implementation...

Orphan entities:
CheckXLogLogicalInfo -> remove (came with 67c20979c [1])
HAVE_ATOMIC_H, HAVE_MBARRIER_H -> remove (per 25f36066d)
#ifdef BS_DEBUG in contrib/ltree/ltxtquery_io.c -> remove (introduced with 1dedbf2da)

The patch is attached for your convenience.

[1] The implementation was removed in v34-0001-Toggle-*.patch:
https://www.postgresql.org/message-id/flat/CAD21AoCVLeLYq09pQPaWs%2BJwdni5FuJ8v2jgq-u9_uFbcp6UbA%40mail.gmail.com

Best regards,
Alexander

  40. Re: Typos in the code and README

    Michael Paquier <michael@paquier.xyz> — 2026-01-04T23:02:22Z

    On Thu, Jan 01, 2026 at 10:00:00AM +0200, Alexander Lakhin wrote:
> READ_WRITE_PARSE_PLAN_TREES -> WRITE_READ_PARSE_PLAN_TREES
> DEFAULT_DEBUG_READ_WRITE_PARSE_PLAN_TREES -> DEFAULT_DEBUG_WRITE_READ_PARSE_PLAN_TREES

I thought that this was a bug first, but we also use this name for the
default value.

> replace_s(..., int * adjustment) -> replace_s(..., int * adjptr)

Updating this one contradicts with the upstream sources, see
b464e51ab32f that has done the opposite switch.

> There is also
> extern int        pgwin32_recv(SOCKET s, char *buf, int len, int flags);
> vs
> pgwin32_recv(SOCKET s, char *buf, int len, int f)
> but I've left at as-is; probably the change should be made in the
> implementation...

Using "flags" would be confusing I think as things stand, the code
casts a DWORD with the same name.

> Orphan entities:
> HAVE_ATOMIC_H, HAVE_MBARRIER_H -> remove (per 25f36066d)

Indeed..  It seems slightly cleaner to remove these separately.  Will
do so.

> #ifdef BS_DEBUG in contrib/ltree/ltxtquery_io.c -> remove (introduced with 1dedbf2da)

I am wondering if BS stands for what it means or if it's something
else..
--
Michael

  41. Re: Typos in the code and README

    Tom Lane <tgl@sss.pgh.pa.us> — 2026-01-04T23:32:14Z

    Michael Paquier <michael@paquier.xyz> writes:
> On Thu, Jan 01, 2026 at 10:00:00AM +0200, Alexander Lakhin wrote:
>> replace_s(..., int * adjustment) -> replace_s(..., int * adjptr)

> Updating this one contradicts with the upstream sources, see
> b464e51ab32f that has done the opposite switch.

It looks like upstream has fixed the problem -- not by making the
two instances match, but by removing the parameter altogether
(see their recent commit 16edd5abb9ce7386e9208b998cd43524480cb801).

Maybe it's time for another sync with upstream?  But if we don't
feel like doing that right now, I think Alexander's proposed patch
is okay.  It won't make the next sync any harder.

			regards, tom lane

  42. Re: Typos in the code and README

    Michael Paquier <michael@paquier.xyz> — 2026-01-05T00:35:37Z

    On Sun, Jan 04, 2026 at 06:32:14PM -0500, Tom Lane wrote:
> It looks like upstream has fixed the problem -- not by making the
> two instances match, but by removing the parameter altogether
> (see their recent commit 16edd5abb9ce7386e9208b998cd43524480cb801).

Ah, you mean as in [1] as mentioned in [2].  I didn't know this part.
A new sync sounds like a good idea.  I am not sure to see a point in
changing the same thing twice, so...

> Maybe it's time for another sync with upstream?

This sounds like a good idea to me.  The last sync has been done in
February 2025, so it could be done in the next couple of weeks.

[1]: https://github.com/snowballstem/snowball.git
[2]: https://snowballstem.org/download.html
--
Michael

  43. Re: Typos in the code and README

    Tom Lane <tgl@sss.pgh.pa.us> — 2026-01-05T00:46:15Z

    Michael Paquier <michael@paquier.xyz> writes:
> On Sun, Jan 04, 2026 at 06:32:14PM -0500, Tom Lane wrote:
>> Maybe it's time for another sync with upstream?

> This sounds like a good idea to me.  The last sync has been done in
> February 2025, so it could be done in the next couple of weeks.

I can look into that tomorrow or so, if there's not objections.
It looks like nothing exciting has gone into snowball in the last
month or so, so right now might be a good time to sync.

			regards, tom lane

  44. Re: Typos in the code and README

    Andres Freund <andres@anarazel.de> — 2026-01-05T17:24:25Z

    Hi,

On 2026-01-01 10:00:00 +0200, Alexander Lakhin wrote:
> bufhdr -> bufHdr

I just ran into this one due to a rebase conflict. Obviously trivial to
resolve. But what's the point of this change?  If you wanted to fix the
arguably wrong name, it'd have to be "buf" or "BufferDesc", as there's no
variable named either bufhdr or bufHdr.

Having random stuff mixed into these changes is one of the reasons why I so
dislike them.

Greetings,

Andres Freund

  45. Re: Typos in the code and README

    Alexander Lakhin <exclusion@gmail.com> — 2026-01-05T19:00:00Z

    Hello Andres,

On Mon, Jan 5, 2026, 21:24 Andres Freund <andres@anarazel.de> wrote:

>
> On 2026-01-01 10:00:00 +0200, Alexander Lakhin wrote:
> > bufhdr -> bufHdr
>
> I just ran into this one due to a rebase conflict. Obviously trivial to
> resolve. But what's the point of this change?  If you wanted to fix the
> arguably wrong name, it'd have to be "buf" or "BufferDesc", as there's no
> variable named either bufhdr or bufHdr.
>

I thought that the initial spelling was meant to reference bufHdr variable
as if it was declared locally as we can see this name in other functions in
bufmgr.c (at 5d508736).

The point was to have no references to entities that you can't find in the
source tree.


> Having random stuff mixed into these changes is one of the reasons why I so
> dislike them.
>

I'm not sure how to arrange such changes to process them efficiently --
they could be split to multiple "Fix a typo" patches, of course.

By the way, I've just noticed that I fixed only one out of two typos in
"parititoned", so it's still spelled wrong: "partititoned". Sorry.

Best regards,
Alexander

>

  46. Re: Typos in the code and README

    Andres Freund <andres@anarazel.de> — 2026-01-05T21:39:42Z

    Hi,

On 2026-01-04 18:32:14 -0500, Tom Lane wrote:
> Michael Paquier <michael@paquier.xyz> writes:
> > On Thu, Jan 01, 2026 at 10:00:00AM +0200, Alexander Lakhin wrote:
> >> replace_s(..., int * adjustment) -> replace_s(..., int * adjptr)
> 
> > Updating this one contradicts with the upstream sources, see
> > b464e51ab32f that has done the opposite switch.
> 
> It looks like upstream has fixed the problem -- not by making the
> two instances match, but by removing the parameter altogether
> (see their recent commit 16edd5abb9ce7386e9208b998cd43524480cb801).
> 
> Maybe it's time for another sync with upstream?  But if we don't
> feel like doing that right now, I think Alexander's proposed patch
> is okay.  It won't make the next sync any harder.

Your recent commit unfortunately doesn't build with meson, due to headers now
not just residing in src/include/snowball/ but also
src/include/snowball/libstemmer/.

The fix is trivial - happy to push it if you want?

Greetings,

Andres

  47. Re: Typos in the code and README

    Tom Lane <tgl@sss.pgh.pa.us> — 2026-01-05T21:43:02Z

    Andres Freund <andres@anarazel.de> writes:
> Your recent commit unfortunately doesn't build with meson, due to headers now
> not just residing in src/include/snowball/ but also
> src/include/snowball/libstemmer/.

Yeah --- but that was true before too.  I don't quite see how
it built under meson before.

			regards, tom lane

  48. Re: Typos in the code and README

    Andres Freund <andres@anarazel.de> — 2026-01-05T21:50:59Z

    Hi,

On 2026-01-05 16:43:02 -0500, Tom Lane wrote:
> Andres Freund <andres@anarazel.de> writes:
> > Your recent commit unfortunately doesn't build with meson, due to headers now
> > not just residing in src/include/snowball/ but also
> > src/include/snowball/libstemmer/.
> 
> Yeah --- but that was true before too.  I don't quite see how
> it built under meson before.

Previously the only thing including snowball/libstemmer/*h files was
src/backend/snowball/dict_snowball.c, which included them relative to
src/includ. But now they are also included from
src/backend/snowball/libstemmer/stem_*.c, just as
> #include "stem_*.h"
thereby requiring include/snowball/libstemmer to be an include path.

Greetings,

Andres Freund

  49. Re: Typos in the code and README

    Tom Lane <tgl@sss.pgh.pa.us> — 2026-01-05T21:55:00Z

    Andres Freund <andres@anarazel.de> writes:
> On 2026-01-05 16:43:02 -0500, Tom Lane wrote:
>> Yeah --- but that was true before too.  I don't quite see how
>> it built under meson before.

> Previously the only thing including snowball/libstemmer/*h files was
> src/backend/snowball/dict_snowball.c, which included them relative to
> src/includ. But now they are also included from
> src/backend/snowball/libstemmer/stem_*.c,

Oh, you're right.  So it worked without any Makefile changes because
the Makefile already had

override CPPFLAGS := -I$(top_srcdir)/src/include/snowball \
	-I$(top_srcdir)/src/include/snowball/libstemmer $(CPPFLAGS)

but meson.build had failed to duplicate that.

			regards, tom lane

  50. Re: Typos in the code and README

    Michael Paquier <michael@paquier.xyz> — 2026-01-06T03:27:29Z

    On Mon, Jan 05, 2026 at 11:00:00PM +0400, Alexander Law wrote:
> I thought that the initial spelling was meant to reference bufHdr variable
> as if it was declared locally as we can see this name in other functions in
> bufmgr.c (at 5d508736).

Yeah, I was under the same impression as yours.  The older pattern
could not be found in the tree, the new one is all spread around.

> By the way, I've just noticed that I fixed only one out of two typos in
> "parititoned", so it's still spelled wrong: "partititoned". Sorry.

Meh.  And here I thought that I read all these diffs correctly.  Will
adjust in a bit.
--
Michael

  51. Re: Typos in the code and README

    Andres Freund <andres@anarazel.de> — 2026-01-06T13:43:59Z

    Hi,

On 2026-01-06 12:27:29 +0900, Michael Paquier wrote:
> On Mon, Jan 05, 2026 at 11:00:00PM +0400, Alexander Law wrote:
> > I thought that the initial spelling was meant to reference bufHdr variable
> > as if it was declared locally as we can see this name in other functions in
> > bufmgr.c (at 5d508736).
> 
> Yeah, I was under the same impression as yours.  The older pattern
> could not be found in the tree, the new one is all spread around.

Sure, but that function doesn't use bufHdr, making the new phrasing just as
confusing as the old one.

Greetings,

Andres

  52. Re: Typos in the code and README

    Alexander Lakhin <exclusion@gmail.com> — 2026-04-19T19:00:00Z

    Hello hackers,

Please look at another set of typos and inconsistencies introduced since
2026-01-01:
AttNumber -> AttrNumber

attytypids, attytypmods -> atttypids, atttypmods

AX handle  -> AH handle

big5_to_mic -> remove
(following 77645d44e)

bk_category -> bkcategory
(per postmaster/proctypelist.h)

BufferLockRelease -> BufferLockReleaseSub

build_index_scan -> build_index_scankeys

cacheoffset -> attcacheoff
(see index_getattr())

containg -> containing

Controlfile -> ControlFile

DataChecksumShmemRequest -> DataChecksumsShmemRequest

DataChecksumsLauncher -> DataChecksumsWorkerLauncherMain
(DataChecksumsWorkerLauncherMain() calls SetDataChecksumsOn(), which calls
EmitProcSignalBarrier())

Datachecksumsworker -> DataChecksumsWorker

dc_fake_temptable, dc_modify_db_result -> remove
(the implementation was removed in 07009121c)

determing -> determining

doublecheck -> double-check

Doublecheck -> Double-check

element_patterns -> path_pattern
(align with the implementation)

endTruncOff, endTruncMemb -> oldestMulti, oldestOffset
(align with the implementation)

expr_nulls[]  -> remove
(exprs_nulls[] was removed in
v2jsonb-0001-Add-support-for-exprs-in-pg_restore_extended.patch [1])

forPortionOfClause -> forPortionOf
(align with the implementation)

HeaptupleSatisfiesMVCC -> HeapTupleSatisfiesMVCC

hintbits -> hint bits

hostsfile -> hosts file

idenifier -> identifier

io_worker_launch_delay -> io_worker_launch_interval
(named io_worker_launch_delay in [2], but this name is absent in
v7-0001-aio-Adjust-I-O-worker-pool-size-automatically.patch attached
there)

load_from_disk_pending -> stashes_ready
(cf. "if (pg_atomic_unlocked_test_flag(&pgsa_state->stashes_ready))"
below)

LockManagerShmemSize -> remove the referencing sentence?
(following 9b5acad3f)

nonprivileged -> non-privileged

option_values -> option_value

oveflows -> overflows

path_pattern_lists -> path_elem_lists

paralell -> parallel

pg_buffercache_mark_dirt_relation -> pg_buffercache_mark_dirty_relation

pgpa_relation_identifier -> pgpa_identifier

pgstat_activity -> pg_stat_activity

pg_statistic.ext_data -> pg_statistic_ext_data

precalculated -> pre-calculated

PredicateLockShmemSize -> remove the referencing sentence?
(following 2e0943a85)

queryID -> Query ID

rel_dst -> cc_rel
(rel_dst existed in
v25-0004-Add-CONCURRENTLY-option-to-REPACK-command.patch [3], but then got
transformed into dest->rel in v26 [4] and then renamed to cc_rel)

relfrozen -> relfrozenxid
(as in other occurrences "relfrozenxid/relminmxid")

replorigin_session_origin -> replorigin_xact_state.origin

ResOwnerReleaseBufferPin -> ResOwnerReleaseBuffer
(see fcb9c977a)

RewriteGraphTable -> rewriteGraphTable

RTEkind -> RTE kind

SetHintBitExt -> SetHintBitsExt

signedess -> signedness

statsare -> stats are

SubPlanInfo -> SubPlanRTInfo

SubRTInfo -> SubPlanRTInfo

subsystemslist.h -> subsystemlist.h

swicthed -> switched

tablespacenames -> tablespace names

TABLE_UPDATE_WAIT -> remove the mention
("#define TABLE_UPDATE_WAIT" existed in
v46-0002-Add-CONCURRENTLY-option-to-REPACK-command.patch, but it was
removed in v47 [6].)

targetExpr -> targetRange
(see the code below)

testcluster -> test cluster

themself -> themselves

tkeys -> rids

truncation_block -> truncation_block_length

tupleTableSlot -> TupleTableSlot

VacAttrStatP -> VacAttrStatsP

XLOGShmemSize -> XLOGShmemRequest
(XLOGShmemRequest modifies wal_buffers)


[1] https://www.postgresql.org/message-id/CADkLM%3Ddu%2BOcctrsTk%2BhZryUGy%3D0OnPep-3LdGzut1nqF391%2BEg%40mail.gmail.com
[2] https://www.postgresql.org/message-id/CA%2BhUKGK%3DvELXFXNj2L%3DvTkof6s_EQzTjYXXrUVwOOW0rahEfVg%40mail.gmail.com
[3] https://www.postgresql.org/message-id/202510301734.pj4uds3mqxx4%40alvherre.pgsql
[4] https://www.postgresql.org/message-id/116433.1764870207%40localhost
[5] https://www.postgresql.org/message-id/202603292157.tnqxaozfjlkh%40alvherre.pgsql
[6] https://www.postgresql.org/message-id/202603311523.iqhng5ljkzpq%40alvherre.pgsql

Best regards,
Alexander

  53. Re: Typos in the code and README

    Alexander Lakhin <exclusion@gmail.com> — 2026-04-20T01:00:01Z

    19.04.2026 22:00, Alexander Lakhin wrote:
> Please look at another set of typos and inconsistencies introduced since
> 2026-01-01:

Sorry, I forgot to attach the patch for your convenience.

Best regards,
Alexander

  54. Re: Typos in the code and README

    Daniil Davydov <3danissimo@gmail.com> — 2026-04-20T06:11:48Z

    Hi,

On Mon, Apr 20, 2026 at 2:00 AM Alexander Lakhin <exclusion@gmail.com> wrote:
>
> Please look at another set of typos and inconsistencies introduced since
> 2026-01-01

I've found a few other typos. Although they are quite old, I think they should
be mentioned here :

1) Typo in tidstore.c
"Find the shared the shared radix tree" --> "Find the shared radix tree".

2) A bit inconsistent comment in twophase.c
```
/*
 * Recreates a state file. This is used in WAL replay and during
 * checkpoint creation.
 *
 * Note: content and len don't include CRC.
 */
static void
RecreateTwoPhaseFile(FullTransactionId fxid, void *content, int len)
```

This comment was introduced in the implementation of the two-phase commit [1].
At that time, this function was actually used during replaying the
XLOG_XACT_PREPARE record. Commit [2] changed that code, so now this function is
used only during checkpoint and restartpoint creation.

If restartpoint creation can be considered as part of the WAL replay, we can
leave this comment as is. Otherwise, I suggest changing it to something like
this :
/*
 * Recreates a state file. This is used during checkpoint/restartpoint creation.
 *
 * Note: content and len don't include CRC.
 */

Another idea - we can stop listing the places of function's usage so that we
don't need to touch this comment later.

[1] d0a89683a3a4dd8e76ef0a99101355999e519df5
[2] 728bd991c3c4389fb39c45dcb0fe57e4a1dccd71

--
Best regards,
Daniil Davydov

  55. Re: Typos in the code and README

    Michael Paquier <michael@paquier.xyz> — 2026-04-21T04:23:35Z

    On Mon, Apr 20, 2026 at 04:00:01AM +0300, Alexander Lakhin wrote:
> Sorry, I forgot to attach the patch for your convenience.

I have a bunch of these lying around, already addressed, but not all.
I'll go incorporate your patch to plug in all the other holes you are
pointing at now.  With that included, I have a nice stack that has
accumulated, so I will merge quickly.  The plan is to do so roughly
every two months, depending on the numbers.
--
Michael