postmaster/postgres merge for testing

Lists: pgsql-hackerspgsql-patches
From: Peter Eisentraut <peter_e(at)gmx(dot)net>
To: pgsql-patches(at)postgresql(dot)org
Subject: postmaster/postgres merge for testing
Date: 2006-01-21 22:49:03
Message-ID: 200601212349.04676.peter_e@gmx.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers pgsql-patches

Attached is a patch that merges postmaster and postgres into just a
postmaster command. I have moved a few things around so it would be
good if someone could test this especially on Windows (just building
and regression test should do it).

(It's a bit weird in that src/backend already contains a postmaster
directory so you can't build a "postmaster" file there. So in the
build tree it's called postmaster_. Feel free to make better
suggestions.)

--
Peter Eisentraut
http://developer.postgresql.org/~petere/

Attachment Content-Type Size
postmaster-merge.patch.bz2 application/x-bzip2 15.9 KB

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Peter Eisentraut <peter_e(at)gmx(dot)net>
Cc: pgsql-patches(at)postgresql(dot)org
Subject: Re: postmaster/postgres merge for testing
Date: 2006-01-22 00:38:01
Message-ID: 2679.1137890281@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers pgsql-patches

Peter Eisentraut <peter_e(at)gmx(dot)net> writes:
> (It's a bit weird in that src/backend already contains a postmaster
> directory so you can't build a "postmaster" file there. So in the
> build tree it's called postmaster_. Feel free to make better
> suggestions.)

backend, maybe? Or just keep calling it postgres at that point.

regards, tom lane


From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Peter Eisentraut <peter_e(at)gmx(dot)net>
Cc: pgsql-hackers(at)postgreSQL(dot)org
Subject: Re: [PATCHES] postmaster/postgres merge for testing
Date: 2006-01-23 16:13:25
Message-ID: 5922.1138032805@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers pgsql-patches

Peter Eisentraut <peter_e(at)gmx(dot)net> writes:
> Attached is a patch that merges postmaster and postgres into just a
> postmaster command.

I had some second thoughts about this, specifically about which
direction do we really want to go in. With this patch, it no longer
really matters what the executable file is named, right? We were both
implicitly assuming that the name should end up being "postmaster",
but I think there's a good case to be made that the right thing to do
is to migrate in the direction of having just one executable named
"postgres". We've seen complaints before that having a daemon named
"postmaster" confuses newbies into thinking it's got something to do
with mail. And it's already the case that the child processes all call
themselves "postgres", which will become even more confusing if there is
no longer any executable named "postgres".

If we went in this direction we'd have to keep the installed
postmaster->postgres symlink for awhile to avoid breaking existing
start scripts, but it could be deprecated and then removed in a few
releases.

Thoughts?

regards, tom lane


From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Peter Eisentraut <peter_e(at)gmx(dot)net>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: [PATCHES] postmaster/postgres merge for testing
Date: 2006-01-23 16:51:36
Message-ID: 1138035096.3294.3.camel@swithin
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers pgsql-patches

On Mon, 2006-01-23 at 11:13 -0500, Tom Lane wrote:
> Peter Eisentraut <peter_e(at)gmx(dot)net> writes:
> > Attached is a patch that merges postmaster and postgres into just a
> > postmaster command.
>
> I had some second thoughts about this, specifically about which
> direction do we really want to go in. With this patch, it no longer
> really matters what the executable file is named, right? We were both
> implicitly assuming that the name should end up being "postmaster",
> but I think there's a good case to be made that the right thing to do
> is to migrate in the direction of having just one executable named
> "postgres". We've seen complaints before that having a daemon named
> "postmaster" confuses newbies into thinking it's got something to do
> with mail. And it's already the case that the child processes all call
> themselves "postgres", which will become even more confusing if there is
> no longer any executable named "postgres".
>
> If we went in this direction we'd have to keep the installed
> postmaster->postgres symlink for awhile to avoid breaking existing
> start scripts, but it could be deprecated and then removed in a few
> releases.
>
> Thoughts?

This is clearly better, IMNSHO. I did wonder about postgresqld or
postgresd or some such - many server programs end in "d" or ".d" to
distinguish them from client programs. But probably just "postgres" is
best.

cheers

andrew


From: "Jim C(dot) Nasby" <jnasby(at)pervasive(dot)com>
To: Andrew Dunstan <andrew(at)dunslane(dot)net>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Peter Eisentraut <peter_e(at)gmx(dot)net>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: [PATCHES] postmaster/postgres merge for testing
Date: 2006-01-23 17:00:51
Message-ID: 20060123170051.GM20182@pervasive.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers pgsql-patches

On Mon, Jan 23, 2006 at 11:51:36AM -0500, Andrew Dunstan wrote:
> This is clearly better, IMNSHO. I did wonder about postgresqld or
> postgresd or some such - many server programs end in "d" or ".d" to
> distinguish them from client programs. But probably just "postgres" is
> best.

Or postgresql if we want to be consistent...
</nitpick>
--
Jim C. Nasby, Sr. Engineering Consultant jnasby(at)pervasive(dot)com
Pervasive Software http://pervasive.com work: 512-231-6117
vcard: http://jim.nasby.net/pervasive.vcf cell: 512-569-9461


From: Marko Kreen <markokr(at)gmail(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Peter Eisentraut <peter_e(at)gmx(dot)net>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: [PATCHES] postmaster/postgres merge for testing
Date: 2006-01-24 08:40:51
Message-ID: e51f66da0601240040u283b11ack1736f5ac2bc86dad@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers pgsql-patches

On 1/23/06, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Peter Eisentraut <peter_e(at)gmx(dot)net> writes:
> > Attached is a patch that merges postmaster and postgres into just a
> > postmaster command.
>
> I had some second thoughts about this, specifically about which
> direction do we really want to go in. With this patch, it no longer
> really matters what the executable file is named, right? We were both
> implicitly assuming that the name should end up being "postmaster",
> but I think there's a good case to be made that the right thing to do
> is to migrate in the direction of having just one executable named
> "postgres". We've seen complaints before that having a daemon named
> "postmaster" confuses newbies into thinking it's got something to do
> with mail. And it's already the case that the child processes all call
> themselves "postgres", which will become even more confusing if there is
> no longer any executable named "postgres".

+1 for 'postgres'.

--
marko


From: Gustavo Tonini <gustavotonini(at)gmail(dot)com>
To: pgsql-hackers(at)postgresql(dot)org
Subject: Re: [PATCHES] postmaster/postgres merge for testing
Date: 2006-01-24 10:17:22
Message-ID: 9c31dd0d0601240217t46f06643l@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers pgsql-patches

pgd?

Gustavo.


From: Mark Kirkwood <markir(at)paradise(dot)net(dot)nz>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Peter Eisentraut <peter_e(at)gmx(dot)net>, pgsql-hackers(at)postgreSQL(dot)org
Subject: Re: [PATCHES] postmaster/postgres merge for testing
Date: 2006-01-24 21:49:08
Message-ID: 43D6A0D4.7020009@paradise.net.nz
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers pgsql-patches

Tom Lane wrote:
> Peter Eisentraut <peter_e(at)gmx(dot)net> writes:
>
>>Attached is a patch that merges postmaster and postgres into just a
>>postmaster command.
>
>
> I had some second thoughts about this, specifically about which
> direction do we really want to go in. With this patch, it no longer
> really matters what the executable file is named, right? We were both
> implicitly assuming that the name should end up being "postmaster",
> but I think there's a good case to be made that the right thing to do
> is to migrate in the direction of having just one executable named
> "postgres". We've seen complaints before that having a daemon named
> "postmaster" confuses newbies into thinking it's got something to do
> with mail. And it's already the case that the child processes all call
> themselves "postgres", which will become even more confusing if there is
> no longer any executable named "postgres".
>
> If we went in this direction we'd have to keep the installed
> postmaster->postgres symlink for awhile to avoid breaking existing
> start scripts, but it could be deprecated and then removed in a few
> releases.
>
> Thoughts?
>
>

+1 postgres (having the executable name matching the default os
superuser and database accounts seems logical).


From: Robert Treat <xzilla(at)users(dot)sourceforge(dot)net>
To: Gustavo Tonini <gustavotonini(at)gmail(dot)com>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: [PATCHES] postmaster/postgres merge for testing
Date: 2006-01-25 17:01:13
Message-ID: 1138208484.2200.38.camel@camel
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers pgsql-patches

On Tue, 2006-01-24 at 05:17, Gustavo Tonini wrote:
> pgd?
>

or taking a page out of apache's book, databased ?

Robert Treat
--
Build A Brighter Lamp :: Linux Apache {middleware} PostgreSQL