From: | Alexander Lakhin <exclusion(at)gmail(dot)com> |
---|---|
To: | David Rowley <dgrowleyml(at)gmail(dot)com> |
Cc: | Daniel Gustafsson <daniel(at)yesql(dot)se>, Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com>, Heikki Linnakangas <hlinnaka(at)iki(dot)fi>, PostgreSQL Developers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: Typos in the code and README |
Date: | 2024-07-04 17:00:01 |
Message-ID: | c1d63754-cb85-2d8a-8409-bde2c4d2d04b@gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
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
Attachment | Content-Type | Size |
---|---|---|
fix-inconsistencies.patch | text/x-patch | 8.4 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Pavel Stehule | 2024-07-04 17:15:55 | Re: Improving PL/Tcl's error context reports |
Previous Message | Sergey Solovev | 2024-07-04 16:45:42 | [PATCH] Add log_transaction setting |