On 1/7/23 18:38, Tom Lane wrote:
"Karl O. Pinc" <k...@karlpinc.com> writes:
This is a review of Peter's 2 patches. I see only 1 small problem.
Looking at the documentation, a "postmaster" in the glossary is
defined as the controlling process. This works; it needs to be called
something. There is still a postmaster.pid (etc.) in the data
directory.
The word "postmaster" (case insensitive) shows up 84 times in the
documentation. I looked at all of these.
Hmm ... I thought this patch was about getting rid of the
admittedly-obsolete installed symlink.
That was my understanding too.
If it's trying to get rid of the "postmaster" terminology for our
parent process, I'm very strongly against that, either as regards to
code or documentation.
+1
--
Joe Conway
PostgreSQL Contributors Team
RDS Open Source Databases
Amazon Web Services: https://aws.amazon.com
