From: | David Rowley <dgrowleyml(at)gmail(dot)com> |
---|---|
To: | Alexander Lakhin <exclusion(at)gmail(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
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-08-12 11:59:31 |
Message-ID: | CAApHDvoYsiFkZpjWeLGejibAXi=T3B70MzFr7z-1EiJ4pVX2hA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
(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(at)gmail(dot)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
Attachment | Content-Type | Size |
---|---|---|
remaining_typos.patch | application/octet-stream | 2.6 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Ajin Cherian | 2024-08-12 12:17:55 | Re: Introduce XID age and inactive timeout based replication slot invalidation |
Previous Message | Tomas Vondra | 2024-08-12 11:53:32 | Re: Add support for (Var op Var) clause in extended MCV statistics |