Re: [HACKERS] pg_ctl and port number detection

2010-12-27 Thread Bruce Momjian
Fujii Masao wrote:
 On Fri, Dec 24, 2010 at 11:47 PM, Bruce Momjian br...@momjian.us wrote:
  Applied.
 
 storage.sgml seems to need to be updated.

Ah, I see that now, thanks.  Patch attached and applied.

-- 
  Bruce Momjian  br...@momjian.ushttp://momjian.us
  EnterpriseDB http://enterprisedb.com

  + It's impossible for everything to be true. +
diff --git a/doc/src/sgml/storage.sgml b/doc/src/sgml/storage.sgml
index 260df3d..cda7f64 100644
*** /tmp/jT4w0b_storage.sgml	Mon Dec 27 15:19:18 2010
--- doc/src/sgml/storage.sgml	Mon Dec 27 15:18:51 2010
*** last started with/entry
*** 116,123 
  
  row
   entryfilenamepostmaster.pid//entry
!  entryA lock file recording the current server PID and shared memory
! segment ID (not present after server shutdown)/entry
  /row
  
  /tbody
--- 116,124 
  
  row
   entryfilenamepostmaster.pid//entry
!  entryA lock file recording the current postmaster process id (PID),
!  cluster data directory, port number, Unix domain socket directory,
!  and shared memory segment ID/entry
  /row
  
  /tbody

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] pg_ctl and port number detection

2010-12-26 Thread Fujii Masao
On Fri, Dec 24, 2010 at 11:47 PM, Bruce Momjian br...@momjian.us wrote:
 Applied.

storage.sgml seems to need to be updated.

Regards,

-- 
Fujii Masao
NIPPON TELEGRAPH AND TELEPHONE CORPORATION
NTT Open Source Software Center

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] pg_ctl and port number detection

2010-12-24 Thread Bruce Momjian
Bruce Momjian wrote:
 Tom Lane wrote:
  Actually, if we're going to do this at all, we should do
  
  pid
  datadir
  port
  socketdir
  ... here be dragons ...
  
  so that pg_ctl doesn't have to assume the server is running with a
  default value of unix_socket_dir.  Not sure what to put in the fourth
  line on Windows though ... maybe just leave it empty?
 
 OK, here is a patch that adds the port number and optionally socket
 directory location to postmaster.pid, and modifies pg_ctl to use that
 information.  I throw an error on using Win32 with pre-9.1 servers
 because we can't get the port number from that file.
 
 This removes some crufty code from pg_ctl and removes dependency on
 serveral user-configurable settings that we added as a work-around.
 
 This will allow pg_ctl -w to work more reliabily than it did in the
 past.

Applied.

-- 
  Bruce Momjian  br...@momjian.ushttp://momjian.us
  EnterpriseDB http://enterprisedb.com

  + It's impossible for everything to be true. +

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] pg_ctl and port number detection

2010-12-22 Thread Bruce Momjian
Tom Lane wrote:
 Actually, if we're going to do this at all, we should do
 
   pid
   datadir
   port
   socketdir
   ... here be dragons ...
 
 so that pg_ctl doesn't have to assume the server is running with a
 default value of unix_socket_dir.  Not sure what to put in the fourth
 line on Windows though ... maybe just leave it empty?

OK, here is a patch that adds the port number and optionally socket
directory location to postmaster.pid, and modifies pg_ctl to use that
information.  I throw an error on using Win32 with pre-9.1 servers
because we can't get the port number from that file.

This removes some crufty code from pg_ctl and removes dependency on
serveral user-configurable settings that we added as a work-around.

This will allow pg_ctl -w to work more reliabily than it did in the
past.

-- 
  Bruce Momjian  br...@momjian.ushttp://momjian.us
  EnterpriseDB http://enterprisedb.com

  + It's impossible for everything to be true. +
diff --git a/doc/src/sgml/ref/pg_ctl-ref.sgml b/doc/src/sgml/ref/pg_ctl-ref.sgml
index 2c01e12..c526e8e 100644
*** /tmp/pgrevert.8079/5dqQCd_pg_ctl-ref.sgml	Wed Dec 22 10:10:23 2010
--- doc/src/sgml/ref/pg_ctl-ref.sgml	Wed Dec 22 10:06:33 2010
*** PostgreSQL documentation
*** 348,368 
 para
  Wait for the startup or shutdown to complete.
  Waiting is the default option for shutdowns, but not startups.
  When waiting for shutdown, commandpg_ctl/command waits for
  the server to remove its acronymPID/acronym file.
! When waiting for startup, commandpg_ctl/command repeatedly
! attempts to connect to the server via applicationpsql/, and
! reports success when this is successful.
! commandpg_ctl/command will attempt to use the proper port for
! applicationpsql/. If the environment variable
! envarPGPORT/envar exists, that is used.  Otherwise,
! commandpg_ctl/command will see if a port has been set in the
! filenamepostgresql.conf/filename file.  If not, it will use the
! default port that productnamePostgreSQL/productname was compiled
! with (5432 by default).
! When waiting, commandpg_ctl/command will
! return an exit code based on the success of the startup
! or shutdown.
 /para
/listitem
   /varlistentry
--- 348,359 
 para
  Wait for the startup or shutdown to complete.
  Waiting is the default option for shutdowns, but not startups.
+ When waiting for startup, commandpg_ctl/command repeatedly
+ attempts to connect to the server.
  When waiting for shutdown, commandpg_ctl/command waits for
  the server to remove its acronymPID/acronym file.
! commandpg_ctl/command returns an exit code based on the
! success of the startup or shutdown.
 /para
/listitem
   /varlistentry
*** PostgreSQL documentation
*** 442,469 
  /listitem
 /varlistentry
  
-varlistentry
- termenvarPGHOST/envar/term
- 
- listitem
-  para
-   Default host name or Unix-domain socket location for xref
-   linkend=app-psql (used when waiting for startup).
-  /para
- /listitem
-/varlistentry
- 
-varlistentry
- termenvarPGPORT/envar/term
- 
- listitem
-  para
-   Default port number for xref linkend=app-psql
-   (used when waiting for startup).
-  /para
- /listitem
-/varlistentry
- 
/variablelist
  
para
--- 433,438 
*** PostgreSQL documentation
*** 506,523 
  /listitem
 /varlistentry
  
-varlistentry
- termfilenamepostgresql.conf/filename/term
- 
- listitem
-  para
-   This file, located in the data directory, is parsed to find the
-   proper port to use with applicationpsql/application
-   when waiting for startup.
-  /para
- /listitem
-/varlistentry
- 
/variablelist
   /refsect1
  
--- 475,480 
diff --git a/src/backend/utils/init/miscinit.c b/src/backend/utils/init/miscinit.c
index 14ed914..deb2d58 100644
*** /tmp/pgrevert.8079/PNev2b_miscinit.c	Wed Dec 22 10:10:23 2010
--- src/backend/utils/init/miscinit.c	Wed Dec 22 10:06:33 2010
***
*** 33,38 
--- 33,39 
  #include mb/pg_wchar.h
  #include miscadmin.h
  #include postmaster/autovacuum.h
+ #include postmaster/postmaster.h
  #include storage/fd.h
  #include storage/ipc.h
  #include storage/pg_shmem.h
*** CreateLockFile(const char *filename, boo
*** 658,664 
  			   bool isDDLock, const char *refName)
  {
  	int			fd;
! 	char		buffer[MAXPGPATH + 100];
  	int			ntries;
  	int			len;
  	int			encoded_pid;
--- 659,665 
  			   bool isDDLock, const char *refName)
  {
  	int			fd;
! 	char		buffer[MAXPGPATH * 2 + 256];
  	int			ntries;
  	int			len;
  	int			encoded_pid;
*** CreateLockFile(const char *filename, 

Re: [HACKERS] pg_ctl and port number detection

2010-12-20 Thread Bruce Momjian
Tom Lane wrote:
 Bruce Momjian br...@momjian.us writes:
  I wonder if we should write the port number as the 4th line in
  postmaster.pid and return in a few major releases and use that.  We
  could fall back and use our existing code if there is no 4th line.
 
 No.  If it goes in, it should go in as the third line.  The shmem key
 data is private to the server --- we do not want external programs
 assuming anything at all about the private part of postmaster.pid.

OK, so you are suggesting having it as a third value on the third line?

10231
/u/pgsql/data
  5432001  45481984 port_here
^
I like that better because it simplifies the test and limits the
possibility of non-atomic multi-line writes.  For Win32, we would just
have the port number because the line is normally empty.
  
-- 
  Bruce Momjian  br...@momjian.ushttp://momjian.us
  EnterpriseDB http://enterprisedb.com

  + It's impossible for everything to be true. +

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] pg_ctl and port number detection

2010-12-20 Thread Tom Lane
Bruce Momjian br...@momjian.us writes:
 Tom Lane wrote:
 No.  If it goes in, it should go in as the third line.  The shmem key
 data is private to the server --- we do not want external programs
 assuming anything at all about the private part of postmaster.pid.

 OK, so you are suggesting having it as a third value on the third line?

   10231
   /u/pgsql/data
 5432001  45481984 port_here
   ^

I'm not sure why you're having such a hard time grasping this concept.
We do not want pg_ctl looking at the shmem key information, not even to
the extent of assuming a particular format for it.  Therefore the port
number has to go before it not after it.  What I'm thinking of is

pid
datadir
port
... here be dragons ...

Actually, if we're going to do this at all, we should do

pid
datadir
port
socketdir
... here be dragons ...

so that pg_ctl doesn't have to assume the server is running with a
default value of unix_socket_dir.  Not sure what to put in the fourth
line on Windows though ... maybe just leave it empty?

regards, tom lane

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] pg_ctl and port number detection

2010-12-20 Thread Andrew Dunstan



On 12/20/2010 12:41 PM, Tom Lane wrote:


Actually, if we're going to do this at all, we should do

pid
datadir
port
socketdir
... here be dragons ...

so that pg_ctl doesn't have to assume the server is running with a
default value of unix_socket_dir.  Not sure what to put in the fourth
line on Windows though ... maybe just leave it empty?




Yes, that seems reasonable.

cheers

andrew

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] pg_ctl and port number detection

2010-12-20 Thread Bruce Momjian
Tom Lane wrote:
 Bruce Momjian br...@momjian.us writes:
  Tom Lane wrote:
  No.  If it goes in, it should go in as the third line.  The shmem key
  data is private to the server --- we do not want external programs
  assuming anything at all about the private part of postmaster.pid.
 
  OK, so you are suggesting having it as a third value on the third line?
 
  10231
  /u/pgsql/data
5432001  45481984 port_here
  ^
 
 I'm not sure why you're having such a hard time grasping this concept.
 We do not want pg_ctl looking at the shmem key information, not even to
 the extent of assuming a particular format for it.  Therefore the port
 number has to go before it not after it.  What I'm thinking of is
 
   pid
   datadir
   port
   ... here be dragons ...
 
 Actually, if we're going to do this at all, we should do
 
   pid
   datadir
   port
   socketdir
   ... here be dragons ...
 
 so that pg_ctl doesn't have to assume the server is running with a
 default value of unix_socket_dir.  Not sure what to put in the fourth
 line on Windows though ... maybe just leave it empty?

OK.  I was hesitant to modify the existing postmaster.pid format and was
trying to just add on the end.  It is certainly easier to put it before
the shared memory stuff.  I will work on a patch.

-- 
  Bruce Momjian  br...@momjian.ushttp://momjian.us
  EnterpriseDB http://enterprisedb.com

  + It's impossible for everything to be true. +

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] pg_ctl and port number detection

2010-12-20 Thread Bruce Momjian
Tom Lane wrote:
 Actually, if we're going to do this at all, we should do
 
   pid
   datadir
   port
   socketdir
   ... here be dragons ...
 
 so that pg_ctl doesn't have to assume the server is running with a
 default value of unix_socket_dir.  Not sure what to put in the fourth
 line on Windows though ... maybe just leave it empty?

I am curious about the use of the socketdir.  What can that be used for?

-- 
  Bruce Momjian  br...@momjian.ushttp://momjian.us
  EnterpriseDB http://enterprisedb.com

  + It's impossible for everything to be true. +

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] pg_ctl and port number detection

2010-12-20 Thread Florian Pflug
On Dec21, 2010, at 03:04 , Bruce Momjian wrote:
 Tom Lane wrote:
 Actually, if we're going to do this at all, we should do
 
  pid
  datadir
  port
  socketdir
  ... here be dragons ...
 
 so that pg_ctl doesn't have to assume the server is running with a
 default value of unix_socket_dir.  Not sure what to put in the fourth
 line on Windows though ... maybe just leave it empty?
 
 I am curious about the use of the socketdir.  What can that be used for?

If it's non-default and you want to connect via unix sockets, just knowing
the port won't help much.

best regards,
Florian Pflug


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] pg_ctl and port number detection

2010-12-20 Thread Bruce Momjian
Florian Pflug wrote:
 On Dec21, 2010, at 03:04 , Bruce Momjian wrote:
  Tom Lane wrote:
  Actually, if we're going to do this at all, we should do
  
 pid
 datadir
 port
 socketdir
 ... here be dragons ...
  
  so that pg_ctl doesn't have to assume the server is running with a
  default value of unix_socket_dir.  Not sure what to put in the fourth
  line on Windows though ... maybe just leave it empty?
  
  I am curious about the use of the socketdir.  What can that be used for?
 
 If it's non-default and you want to connect via unix sockets, just knowing
 the port won't help much.

Ah, so pg_ctl is going to use that information too --- good point!

-- 
  Bruce Momjian  br...@momjian.ushttp://momjian.us
  EnterpriseDB http://enterprisedb.com

  + It's impossible for everything to be true. +

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] pg_ctl and port number detection

2010-12-19 Thread Florian Pflug
On Dec19, 2010, at 00:54 , Bruce Momjian wrote:
 I wonder if we should write the port number as the 4th line in
 postmaster.pid and return in a few major releases and use that.  We
 could fall back and use our existing code if there is no 4th line.

What if the postmaster instead created a second unix socket in its
data directory? For security reason, it'd probably need to set
the permissions to 0600, but it'd still allow maintenance tools to
connect reliably if they only knew the data directory.

Don't know if that'd work on windows, though - I have no idea if
we even support something similar to unix sockets there, and if so,
it it lives in the filesystem.

best regards,
Florian Pflug

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] pg_ctl and port number detection

2010-12-19 Thread Magnus Hagander
On Sun, Dec 19, 2010 at 20:16, Florian Pflug f...@phlo.org wrote:
 On Dec19, 2010, at 00:54 , Bruce Momjian wrote:
 I wonder if we should write the port number as the 4th line in
 postmaster.pid and return in a few major releases and use that.  We
 could fall back and use our existing code if there is no 4th line.

 What if the postmaster instead created a second unix socket in its
 data directory? For security reason, it'd probably need to set
 the permissions to 0600, but it'd still allow maintenance tools to
 connect reliably if they only knew the data directory.

 Don't know if that'd work on windows, though - I have no idea if
 we even support something similar to unix sockets there, and if so,
 it it lives in the filesystem.

We don't, and AFAIK there's nothing that lives in the filesystem. You
have named pipes that live in the namespace, but not within
directories in the filesystem.

-- 
 Magnus Hagander
 Me: http://www.hagander.net/
 Work: http://www.redpill-linpro.com/

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] pg_ctl and port number detection

2010-12-19 Thread Florian Pflug
On Dec19, 2010, at 21:10 , Magnus Hagander wrote:
 On Sun, Dec 19, 2010 at 20:16, Florian Pflug f...@phlo.org wrote:
 On Dec19, 2010, at 00:54 , Bruce Momjian wrote:
 I wonder if we should write the port number as the 4th line in
 postmaster.pid and return in a few major releases and use that.  We
 could fall back and use our existing code if there is no 4th line.
 
 What if the postmaster instead created a second unix socket in its
 data directory? For security reason, it'd probably need to set
 the permissions to 0600, but it'd still allow maintenance tools to
 connect reliably if they only knew the data directory.
 
 Don't know if that'd work on windows, though - I have no idea if
 we even support something similar to unix sockets there, and if so,
 it it lives in the filesystem.
 
 We don't, and AFAIK there's nothing that lives in the filesystem. You
 have named pipes that live in the namespace, but not within
 directories in the filesystem.


Hm, OK, that pretty much kills the idea. Having to special-case
windows seems less appealing than the other options.

best regards,
Florian Pflug


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] pg_ctl and port number detection

2010-12-19 Thread Bruce Momjian
Bruce Momjian wrote:
 Andrew Dunstan wrote:
  
  
  On 12/18/2010 06:23 PM, Bruce Momjian wrote:
  
   If you really think that pulling a port number out of the pid file is an
   improvement over what pg_ctl does now, then you need to start by storing
   the port number, as such, in the pid file.  Not something that might or
   might not be related to the port number.  But what we have to discuss
   before that is whether we mind having a significant postmaster version
   dependency in pg_ctl.
   OK, good point on the version issue.  Let's see if we get more
   complaints before changing this.  Thanks.
  
  
  Wasn't there a proposal to provide an explicit port parameter to pg_ctl, 
  instead of relying on PGPORT? That would probably be a small advance.
 
 I do not remember that suggestion.
 
 I wonder if we should write the port number as the 4th line in
 postmaster.pid and return in a few major releases and use that.  We
 could fall back and use our existing code if there is no 4th line.

OK, here is a modified idea.  For 9.1, we have the postmaster write the
port number as the fourth line in postmaster.pid.  pg_ctl will use that
fourth line if it exists, i.e. the postmaster is 9.1+.

If the fourth line is missing, we use the first number of the third line
on Unix and divide that by 1000 to get the port number.  That file
format is not going to change because it is pre-9.1.  If the third line
is empty (e.g. Windows) we either use PGPORT or throw an error.

So, we have a fine solution for 9.1+ servers, and all Unix servers, and
if you want to use a 9.1+ pg_ctl on a pre-9.1 server on Windows and use
the -w flag and a non-standard port number, you must specify PGPORT. 
Based on the fact that most Windows users use the one-click installer
that will use the matching pg_ctl version, it seems this will work just
fine.

-- 
  Bruce Momjian  br...@momjian.ushttp://momjian.us
  EnterpriseDB http://enterprisedb.com

  + It's impossible for everything to be true. +

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] pg_ctl and port number detection

2010-12-19 Thread Tom Lane
Bruce Momjian br...@momjian.us writes:
 I wonder if we should write the port number as the 4th line in
 postmaster.pid and return in a few major releases and use that.  We
 could fall back and use our existing code if there is no 4th line.

No.  If it goes in, it should go in as the third line.  The shmem key
data is private to the server --- we do not want external programs
assuming anything at all about the private part of postmaster.pid.

regards, tom lane

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] pg_ctl and port number detection

2010-12-18 Thread Tom Lane
Bruce Momjian br...@momjian.us writes:
 pg_ctl.c::test_postmaster_connection() has some fragile code that tries
 to detect the server port number by looking in the pg_ctl -o string,

It may be fragile, but it works; or at least I've not heard complaints
about it lately.

 I think a simpler solution would be to look in postmaster.pid:
 pg_ctl already knows the data directory.  If the file is missing, the
 server is not running.  If the file exists, the first number on the last
 line, divided by 1000, is the port number.

That's somewhere between fragile and outright wrong.

regards, tom lane

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] pg_ctl and port number detection

2010-12-18 Thread Bruce Momjian
Tom Lane wrote:
 Bruce Momjian br...@momjian.us writes:
  pg_ctl.c::test_postmaster_connection() has some fragile code that tries
  to detect the server port number by looking in the pg_ctl -o string,
 
 It may be fragile, but it works; or at least I've not heard complaints
 about it lately.

True.

  I think a simpler solution would be to look in postmaster.pid:
  pg_ctl already knows the data directory.  If the file is missing, the
  server is not running.  If the file exists, the first number on the last
  line, divided by 1000, is the port number.
 
 That's somewhere between fragile and outright wrong.

Please explain why my idea is not an improvement.

-- 
  Bruce Momjian  br...@momjian.ushttp://momjian.us
  EnterpriseDB http://enterprisedb.com

  + It's impossible for everything to be true. +

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] pg_ctl and port number detection

2010-12-18 Thread Tom Lane
Bruce Momjian br...@momjian.us writes:
 Tom Lane wrote:
 Bruce Momjian br...@momjian.us writes:
 pg_ctl already knows the data directory.  If the file is missing, the
 server is not running.  If the file exists, the first number on the last
 line, divided by 1000, is the port number.

 That's somewhere between fragile and outright wrong.

 Please explain why my idea is not an improvement.

Because it's assuming that those numbers are sysv shmem keys derived in
a particular way.  We have platforms on which that is wrong, Windows
being the most obvious example.  Reading the shmem key assignment code
closely will suggest to you other ways that this could fail.  Not to
mention that people propose getting rid of sysv shmem approximately
every other month, and perhaps someday that will actually happen;
whereupon whatever might get logged in postmaster.pid could be something
completely different.

If you really think that pulling a port number out of the pid file is an
improvement over what pg_ctl does now, then you need to start by storing
the port number, as such, in the pid file.  Not something that might or
might not be related to the port number.  But what we have to discuss
before that is whether we mind having a significant postmaster version
dependency in pg_ctl.

regards, tom lane

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] pg_ctl and port number detection

2010-12-18 Thread Bruce Momjian
Tom Lane wrote:
 Bruce Momjian br...@momjian.us writes:
  Tom Lane wrote:
  Bruce Momjian br...@momjian.us writes:
  pg_ctl already knows the data directory.  If the file is missing, the
  server is not running.  If the file exists, the first number on the last
  line, divided by 1000, is the port number.
 
  That's somewhere between fragile and outright wrong.
 
  Please explain why my idea is not an improvement.
 
 Because it's assuming that those numbers are sysv shmem keys derived in
 a particular way.  We have platforms on which that is wrong, Windows
 being the most obvious example.  Reading the shmem key assignment code
 closely will suggest to you other ways that this could fail.  Not to
 mention that people propose getting rid of sysv shmem approximately
 every other month, and perhaps someday that will actually happen;
 whereupon whatever might get logged in postmaster.pid could be something
 completely different.

Yeah, I was afraid of Windows.

 If you really think that pulling a port number out of the pid file is an
 improvement over what pg_ctl does now, then you need to start by storing
 the port number, as such, in the pid file.  Not something that might or
 might not be related to the port number.  But what we have to discuss
 before that is whether we mind having a significant postmaster version
 dependency in pg_ctl.

OK, good point on the version issue.  Let's see if we get more
complaints before changing this.  Thanks.

-- 
  Bruce Momjian  br...@momjian.ushttp://momjian.us
  EnterpriseDB http://enterprisedb.com

  + It's impossible for everything to be true. +

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] pg_ctl and port number detection

2010-12-18 Thread Andrew Dunstan



On 12/18/2010 06:23 PM, Bruce Momjian wrote:



If you really think that pulling a port number out of the pid file is an
improvement over what pg_ctl does now, then you need to start by storing
the port number, as such, in the pid file.  Not something that might or
might not be related to the port number.  But what we have to discuss
before that is whether we mind having a significant postmaster version
dependency in pg_ctl.

OK, good point on the version issue.  Let's see if we get more
complaints before changing this.  Thanks.



Wasn't there a proposal to provide an explicit port parameter to pg_ctl, 
instead of relying on PGPORT? That would probably be a small advance.


cheers

andrew

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] pg_ctl and port number detection

2010-12-18 Thread Bruce Momjian
Andrew Dunstan wrote:
 
 
 On 12/18/2010 06:23 PM, Bruce Momjian wrote:
 
  If you really think that pulling a port number out of the pid file is an
  improvement over what pg_ctl does now, then you need to start by storing
  the port number, as such, in the pid file.  Not something that might or
  might not be related to the port number.  But what we have to discuss
  before that is whether we mind having a significant postmaster version
  dependency in pg_ctl.
  OK, good point on the version issue.  Let's see if we get more
  complaints before changing this.  Thanks.
 
 
 Wasn't there a proposal to provide an explicit port parameter to pg_ctl, 
 instead of relying on PGPORT? That would probably be a small advance.

I do not remember that suggestion.

I wonder if we should write the port number as the 4th line in
postmaster.pid and return in a few major releases and use that.  We
could fall back and use our existing code if there is no 4th line.

-- 
  Bruce Momjian  br...@momjian.ushttp://momjian.us
  EnterpriseDB http://enterprisedb.com

  + It's impossible for everything to be true. +

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers