Re: [PATCHES] uptime function to postmaster
Neil Conway wrote: > Bruce Momjian wrote: > > We need to preceed our function names with pg_ for cases like this where > > we are supplying pg-specific behavior. > > We do? I'm not sure I can see much of a consistent naming convention for > functions like these: version(), obj_description(), has_xyz_privilege(), > format_type(), set_config(), and the like are surely "pg-specific", for > example. Yea, seems we aren't consistent. -- Bruce Momjian| http://candle.pha.pa.us pgman@candle.pha.pa.us | (610) 359-1001 + If your life is a hard drive, | 13 Roberts Road + Christ can be your backup.| Newtown Square, Pennsylvania 19073 ---(end of broadcast)--- TIP 1: subscribe and unsubscribe commands go to [EMAIL PROTECTED]
Re: [PATCHES] uptime function to postmaster
Bruce Momjian wrote: We need to preceed our function names with pg_ for cases like this where we are supplying pg-specific behavior. We do? I'm not sure I can see much of a consistent naming convention for functions like these: version(), obj_description(), has_xyz_privilege(), format_type(), set_config(), and the like are surely "pg-specific", for example. -Neil ---(end of broadcast)--- TIP 2: you can get off all lists at once with the unregister command (send "unregister YourEmailAddressHere" to [EMAIL PROTECTED])
Re: [PATCHES] uptime function to postmaster
Euler Taveira de Oliveira wrote: > Hi Bruce, > > > I think we should return intervals only when we can't return > > meaningful > > timestamp values. I don't have any logic to back up that opinion, > > though. > > > Agreed. > > > > > We need to preceed our function names with pg_ for cases like this > > where > > we are supplying pg-specific behavior. > > > Agreed. > > An updated version of the patch is attached. It is just implement > 'pg_start_time' function that works in multi-user and stand-alone. Docs > is attached too. I have applied the attached patch, calling the function pg_postmaster_start_time(). I realize a stand-alone backend doesn't have a postmaster, but this is probably as clear as we are going to get. Do we want this to be executed only by super-users? I know there was some discussion about that but I didn't see a conclusion. The only argument I heard was something about random seeds, but that seemed like a weak argument. -- Bruce Momjian| http://candle.pha.pa.us pgman@candle.pha.pa.us | (610) 359-1001 + If your life is a hard drive, | 13 Roberts Road + Christ can be your backup.| Newtown Square, Pennsylvania 19073 Index: doc/src/sgml/func.sgml === RCS file: /cvsroot/pgsql/doc/src/sgml/func.sgml,v retrieving revision 1.254 diff -c -c -r1.254 func.sgml *** doc/src/sgml/func.sgml 13 Jun 2005 02:40:04 - 1.254 --- doc/src/sgml/func.sgml 14 Jun 2005 20:59:17 - *** *** 8120,8125 --- 8120,8131 +pg_postmaster_start_time() +timestamp with time zone +postmaster start time + + + user name equivalent to current_user *** *** 8217,8222 --- 8223,8237 + pg_postmaster_start_time + + + + pg_postmaster_start_time() returns the timestamp with time zone + when the postmaster started. + + + version Index: src/backend/postmaster/postmaster.c === RCS file: /cvsroot/pgsql/src/backend/postmaster/postmaster.c,v retrieving revision 1.452 diff -c -c -r1.452 postmaster.c *** src/backend/postmaster/postmaster.c 9 Jun 2005 22:01:12 - 1.452 --- src/backend/postmaster/postmaster.c 14 Jun 2005 20:59:21 - *** *** 222,227 --- 222,230 bool ClientAuthInProgress = false; /* T during new-client * authentication */ + /* Backend startup time */ + TimestampTz StartTime; + /* * State for assigning random salts and cancel keys. * Also, the global MyCancelKey passes the cancel key assigned to a given *** *** 330,335 --- 333,339 InheritableSocket pgStatPipe0; InheritableSocket pgStatPipe1; pid_t PostmasterPid; + TimestampTz StartTime; #ifdef WIN32 HANDLE PostmasterHandle; HANDLE initial_signal_pipe; *** *** 372,377 --- 376,384 char *userDoption = NULL; int i; + AbsoluteTimeStartTimeSec; /* integer part */ + int StartTimeUSec; /* microsecond part */ + /* This will call exit() if strdup() fails. */ progname = get_progname(argv[0]); *** *** 914,919 --- 921,932 */ StartupPID = StartupDataBase(); + /* +* Get start up time +*/ + StartTimeSec = GetCurrentAbsoluteTimeUsec(&StartTimeUSec); + StartTime = AbsoluteTimeUsecToTimestampTz(StartTimeSec, StartTimeUSec); + status = ServerLoop(); /* *** *** 3603,3608 --- 3616,3622 write_inheritable_socket(¶m->pgStatPipe1, pgStatPipe[1], childPid); param->PostmasterPid = PostmasterPid; + param->StartTime = StartTime; #ifdef WIN32 param->PostmasterHandle = PostmasterHandle; *** *** 3805,3810 --- 3819,3825 read_inheritable_socket(&pgStatPipe[1], ¶m->pgStatPipe1); PostmasterPid = param->PostmasterPid; + StartTime = param->StartTime; #ifdef WIN32 PostmasterHandle = param->PostmasterHandle; Index: src/backend/tcop/postgres.c === RCS file: /cvsroot/pgsql/src/backend/tcop/postgres.c,v retrieving revision 1.447 diff -c -c -r1.447 postgres.c *** src/backend/tcop/postgres.c 3 Jun 2005 23:05:29 - 1.447 --- src/backend/tcop/postgres.c 14 Jun 2005 20:59:22 - *** *** 149,154 --- 149,157 #endif /* TCOP_DONTUSENEWLINE */ + /* Backend startup time */ + TimestampTz StartTime; + /* -
Re: [PATCHES] uptime function to postmaster
I think we agreed on pg_postmaster_start_time() so it is clear it is the postmaster, and not the session. Your patch has been added to the PostgreSQL unapplied patches list at: http://momjian.postgresql.org/cgi-bin/pgpatches It will be applied as soon as one of the PostgreSQL committers reviews and approves it. --- Euler Taveira de Oliveira wrote: > Hi Bruce, > > > I think we should return intervals only when we can't return > > meaningful > > timestamp values. I don't have any logic to back up that opinion, > > though. > > > Agreed. > > > > > We need to preceed our function names with pg_ for cases like this > > where > > we are supplying pg-specific behavior. > > > Agreed. > > An updated version of the patch is attached. It is just implement > 'pg_start_time' function that works in multi-user and stand-alone. Docs > is attached too. > > Comments? > > > Euler Taveira de Oliveira > euler[at]yahoo_com_br > > __ > Converse com seus amigos em tempo real com o Yahoo! Messenger > http://br.download.yahoo.com/messenger/ Content-Description: 958887538-uptime6.diff [ Attachment, skipping... ] > > ---(end of broadcast)--- > TIP 2: you can get off all lists at once with the unregister command > (send "unregister YourEmailAddressHere" to [EMAIL PROTECTED]) -- Bruce Momjian| http://candle.pha.pa.us pgman@candle.pha.pa.us | (610) 359-1001 + If your life is a hard drive, | 13 Roberts Road + Christ can be your backup.| Newtown Square, Pennsylvania 19073 ---(end of broadcast)--- TIP 1: subscribe and unsubscribe commands go to [EMAIL PROTECTED]
Re: [PATCHES] uptime function to postmaster
Hi Bruce, > I think we should return intervals only when we can't return > meaningful > timestamp values. I don't have any logic to back up that opinion, > though. > Agreed. > > We need to preceed our function names with pg_ for cases like this > where > we are supplying pg-specific behavior. > Agreed. An updated version of the patch is attached. It is just implement 'pg_start_time' function that works in multi-user and stand-alone. Docs is attached too. Comments? Euler Taveira de Oliveira euler[at]yahoo_com_br __ Converse com seus amigos em tempo real com o Yahoo! Messenger http://br.download.yahoo.com/messenger/ uptime6.diff Description: 958887538-uptime6.diff ---(end of broadcast)--- TIP 2: you can get off all lists at once with the unregister command (send "unregister YourEmailAddressHere" to [EMAIL PROTECTED])
Re: [PATCHES] uptime function to postmaster
Bruce Momjian writes: > I think we should return intervals only when we can't return meaningful > timestamp values. I don't have any logic to back up that opinion, though. It's easy: a value measured as an interval will be obsolete by the time it's delivered to the client. A start timestamp is actually meaningful information that will still be correct when used; uptime is fragile. regards, tom lane ---(end of broadcast)--- TIP 4: Don't 'kill -9' the postmaster
Re: [PATCHES] uptime function to postmaster
Euler Taveira de Oliveira wrote: > Hi Bruce, > > > > > I think we are best with just pg_startime. If people want the > > interval > > they can subtract it from CURRENT_TIMESTAMP. I have added Matthias's > > version to the patch queue. > > > > > OK. But IIRC the Matthias implementation doesn't work in standalone > mode. And talking about the 'interval', I think it's too ugly make > this: > select CURRENT_TIMESTAMP - pg_starttime(); > > Isn't it more simple do this? > select pg_uptime(); I think we should return intervals only when we can't return meaningful timestamp values. I don't have any logic to back up that opinion, though. > > I think few people will use start_time and more people will use uptime > that's why I propose the 'uptime' function. We need to preceed our function names with pg_ for cases like this where we are supplying pg-specific behavior. > Talking abouts names, IMHO we need to go with uptime() and > start_time(). Why? That's because a system function and it's about > server. When we implement backend uptime, we can go with > connection_uptime() and connection_start_time(). -- Bruce Momjian| http://candle.pha.pa.us pgman@candle.pha.pa.us | (610) 359-1001 + If your life is a hard drive, | 13 Roberts Road + Christ can be your backup.| Newtown Square, Pennsylvania 19073 ---(end of broadcast)--- TIP 5: Have you checked our extensive FAQ? http://www.postgresql.org/docs/faq
Re: [PATCHES] uptime function to postmaster
Hi Bruce, > > I think we are best with just pg_startime. If people want the > interval > they can subtract it from CURRENT_TIMESTAMP. I have added Matthias's > version to the patch queue. > > OK. But IIRC the Matthias implementation doesn't work in standalone mode. And talking about the 'interval', I think it's too ugly make this: select CURRENT_TIMESTAMP - pg_starttime(); Isn't it more simple do this? select pg_uptime(); I think few people will use start_time and more people will use uptime that's why I propose the 'uptime' function. Talking abouts names, IMHO we need to go with uptime() and start_time(). Why? That's because a system function and it's about server. When we implement backend uptime, we can go with connection_uptime() and connection_start_time(). Comments? Euler Taveira de Oliveira euler[at]yahoo_com_br __ Converse com seus amigos em tempo real com o Yahoo! Messenger http://br.download.yahoo.com/messenger/ ---(end of broadcast)--- TIP 7: don't forget to increase your free space map settings
Re: [PATCHES] uptime function to postmaster
I think we are best with just pg_startime. If people want the interval they can subtract it from CURRENT_TIMESTAMP. I have added Matthias's version to the patch queue. --- Euler Taveira de Oliveira wrote: > > Bruce Momjian wrote: > > > > This has been saved for the 8.1 release: > > > > http:/momjian.postgresql.org/cgi-bin/pgpatches2 > > > > > --- > > > > Hi, > > I redo this patch adding the funcionality that Matthias implemented > (starttime). Basically I changed the uptime()'s return type to > 'interval' (more funcional now, uh?) and rework in the Matthias > function (start_time()). The last one return type is 'timestamp with > time zone'. The docs are attached to, but maybe need some > work on it. > > Comments? > > > > = > Euler Taveira de Oliveira > euler[at]yahoo_com_br > > > > > > ___ > Yahoo! Acesso Gr?tis - Instale o discador do Yahoo! agora. > http://br.acesso.yahoo.com/ - Internet r?pida e gr?tis Content-Description: uptime5.diff > *** ./doc/src/sgml/func.sgml.orig 2005-01-20 18:23:48.0 -0200 > --- ./doc/src/sgml/func.sgml 2005-01-20 16:44:52.0 -0200 > *** > *** 8060,8065 > --- 8060,8077 > > > > +start_time() > +timestamp with time zone > +PostgreSQL startup date and time > + > + > + > +uptime() > +interval > +PostgreSQL uptime information > + > + > + > user > name > equivalent to current_user > *** > *** 8157,8162 > --- 8169,8192 > > > > + start_time > + > + > + > + start_time() returns the timestamp with time zone > + which the PostgreSQL was started. > + > + > + > + uptime > + > + > + > + uptime() returns the PostgreSQL > + uptime information. > + > + > + > version > > > *** ./src/backend/postmaster/postmaster.c.orig2005-01-20 > 18:24:36.0 -0200 > --- ./src/backend/postmaster/postmaster.c 2005-01-20 16:44:52.0 > -0200 > *** > *** 221,226 > --- 221,229 > boolClientAuthInProgress = false; /* T during > new-client > > * authentication */ > > + /* Backend startup time */ > + TimestampTz StartTime; > + > /* >* State for assigning random salts and cancel keys. >* Also, the global MyCancelKey passes the cancel key assigned to a given > *** > *** 329,334 > --- 332,338 > InheritableSocket pgStatPipe0; > InheritableSocket pgStatPipe1; > pid_t PostmasterPid; > + TimestampTz StartTime; > #ifdef WIN32 > HANDLE PostmasterHandle; > HANDLE initial_signal_pipe; > *** > *** 371,376 > --- 375,383 > char *userDoption = NULL; > int i; > > + AbsoluteTimeStartTimeSec; /* integer part */ > + int StartTimeUSec; /* microsecond part */ > + > /* This will call exit() if strdup() fails. */ > progname = get_progname(argv[0]); > > *** > *** 915,920 > --- 922,933 >*/ > StartupPID = StartupDataBase(); > > + /* > + * Get start up time > + */ > + StartTimeSec = GetCurrentAbsoluteTimeUsec(&StartTimeUSec); > + StartTime = AbsoluteTimeUsecToTimestampTz(StartTimeSec, StartTimeUSec); > + > status = ServerLoop(); > > /* > *** > *** 3669,3674 > --- 3682,3688 > write_inheritable_socket(¶m->pgStatPipe1, pgStatPipe[1], childPid); > > param->PostmasterPid = PostmasterPid; > + param->StartTime = StartTime; > > #ifdef WIN32 > param->PostmasterHandle = PostmasterHandle; > *** > *** 3871,3876 > --- 3885,3891 > read_inheritable_socket(&pgStatPipe[1], ¶m->pgStatPipe1); > > PostmasterPid = param->PostmasterPid; > + StartTime = param->StartTime; > > #ifdef WIN32 > PostmasterHandle = param->PostmasterHandle; > *** ./src/backend/tcop/postgres.c.orig2005-01-20 18:39:18.0 > -0200 > --- ./src/backend/tcop/postgres.c 2005-01-20 18:15:07.0 -0200 > *** > *** 144,149 > --- 144,152 > #endif /* TCOP_DONTUSENEWLINE */ > > > + /* Backend startup time */ > + TimestampTz StartTime; > + > /* >* decls for routines only used in this file >* > *** > *** 2217, ***
Re: [PATCHES] uptime function to postmaster
This has been saved for the 8.1 release: http://momjian.postgresql.org/cgi-bin/pgpatches2 --- Euler Taveira de Oliveira wrote: > > Bruce Momjian wrote: > > > > This has been saved for the 8.1 release: > > > > http:/momjian.postgresql.org/cgi-bin/pgpatches2 > > > > > --- > > > > Hi, > > I redo this patch adding the funcionality that Matthias implemented > (starttime). Basically I changed the uptime()'s return type to > 'interval' (more funcional now, uh?) and rework in the Matthias > function (start_time()). The last one return type is 'timestamp with > time zone'. The docs are attached to, but maybe need some > work on it. > > Comments? > > > > = > Euler Taveira de Oliveira > euler[at]yahoo_com_br > > > > > > ___ > Yahoo! Acesso Gr?tis - Instale o discador do Yahoo! agora. > http://br.acesso.yahoo.com/ - Internet r?pida e gr?tis Content-Description: uptime5.diff > *** ./doc/src/sgml/func.sgml.orig 2005-01-20 18:23:48.0 -0200 > --- ./doc/src/sgml/func.sgml 2005-01-20 16:44:52.0 -0200 > *** > *** 8060,8065 > --- 8060,8077 > > > > +start_time() > +timestamp with time zone > +PostgreSQL startup date and time > + > + > + > +uptime() > +interval > +PostgreSQL uptime information > + > + > + > user > name > equivalent to current_user > *** > *** 8157,8162 > --- 8169,8192 > > > > + start_time > + > + > + > + start_time() returns the timestamp with time zone > + which the PostgreSQL was started. > + > + > + > + uptime > + > + > + > + uptime() returns the PostgreSQL > + uptime information. > + > + > + > version > > > *** ./src/backend/postmaster/postmaster.c.orig2005-01-20 > 18:24:36.0 -0200 > --- ./src/backend/postmaster/postmaster.c 2005-01-20 16:44:52.0 > -0200 > *** > *** 221,226 > --- 221,229 > boolClientAuthInProgress = false; /* T during > new-client > > * authentication */ > > + /* Backend startup time */ > + TimestampTz StartTime; > + > /* >* State for assigning random salts and cancel keys. >* Also, the global MyCancelKey passes the cancel key assigned to a given > *** > *** 329,334 > --- 332,338 > InheritableSocket pgStatPipe0; > InheritableSocket pgStatPipe1; > pid_t PostmasterPid; > + TimestampTz StartTime; > #ifdef WIN32 > HANDLE PostmasterHandle; > HANDLE initial_signal_pipe; > *** > *** 371,376 > --- 375,383 > char *userDoption = NULL; > int i; > > + AbsoluteTimeStartTimeSec; /* integer part */ > + int StartTimeUSec; /* microsecond part */ > + > /* This will call exit() if strdup() fails. */ > progname = get_progname(argv[0]); > > *** > *** 915,920 > --- 922,933 >*/ > StartupPID = StartupDataBase(); > > + /* > + * Get start up time > + */ > + StartTimeSec = GetCurrentAbsoluteTimeUsec(&StartTimeUSec); > + StartTime = AbsoluteTimeUsecToTimestampTz(StartTimeSec, StartTimeUSec); > + > status = ServerLoop(); > > /* > *** > *** 3669,3674 > --- 3682,3688 > write_inheritable_socket(¶m->pgStatPipe1, pgStatPipe[1], childPid); > > param->PostmasterPid = PostmasterPid; > + param->StartTime = StartTime; > > #ifdef WIN32 > param->PostmasterHandle = PostmasterHandle; > *** > *** 3871,3876 > --- 3885,3891 > read_inheritable_socket(&pgStatPipe[1], ¶m->pgStatPipe1); > > PostmasterPid = param->PostmasterPid; > + StartTime = param->StartTime; > > #ifdef WIN32 > PostmasterHandle = param->PostmasterHandle; > *** ./src/backend/tcop/postgres.c.orig2005-01-20 18:39:18.0 > -0200 > --- ./src/backend/tcop/postgres.c 2005-01-20 18:15:07.0 -0200 > *** > *** 144,149 > --- 144,152 > #endif /* TCOP_DONTUSENEWLINE */ > > > + /* Backend startup time */ > + TimestampTz StartTime; > + > /* >* decls for routines only used in this file >* > *** > *** 2217, > --- 2220,2228 > sigjmp_buf local_sigjmp_buf; >
Re: [PATCHES] uptime function to postmaster
Bruce Momjian wrote: > > This has been saved for the 8.1 release: > > http:/momjian.postgresql.org/cgi-bin/pgpatches2 > > --- > Hi, I redo this patch adding the funcionality that Matthias implemented (starttime). Basically I changed the uptime()'s return type to 'interval' (more funcional now, uh?) and rework in the Matthias function (start_time()). The last one return type is 'timestamp with time zone'. The docs are attached to, but maybe need some work on it. Comments? = Euler Taveira de Oliveira euler[at]yahoo_com_br ___ Yahoo! Acesso Grátis - Instale o discador do Yahoo! agora. http://br.acesso.yahoo.com/ - Internet rápida e grátis*** ./doc/src/sgml/func.sgml.orig 2005-01-20 18:23:48.0 -0200 --- ./doc/src/sgml/func.sgml2005-01-20 16:44:52.0 -0200 *** *** 8060,8065 --- 8060,8077 +start_time() +timestamp with time zone +PostgreSQL startup date and time + + + +uptime() +interval +PostgreSQL uptime information + + + user name equivalent to current_user *** *** 8157,8162 --- 8169,8192 + start_time + + + + start_time() returns the timestamp with time zone + which the PostgreSQL was started. + + + + uptime + + + + uptime() returns the PostgreSQL + uptime information. + + + version *** ./src/backend/postmaster/postmaster.c.orig 2005-01-20 18:24:36.0 -0200 --- ./src/backend/postmaster/postmaster.c 2005-01-20 16:44:52.0 -0200 *** *** 221,226 --- 221,229 bool ClientAuthInProgress = false; /* T during new-client * authentication */ + /* Backend startup time */ + TimestampTz StartTime; + /* * State for assigning random salts and cancel keys. * Also, the global MyCancelKey passes the cancel key assigned to a given *** *** 329,334 --- 332,338 InheritableSocket pgStatPipe0; InheritableSocket pgStatPipe1; pid_t PostmasterPid; + TimestampTz StartTime; #ifdef WIN32 HANDLE PostmasterHandle; HANDLE initial_signal_pipe; *** *** 371,376 --- 375,383 char *userDoption = NULL; int i; + AbsoluteTimeStartTimeSec; /* integer part */ + int StartTimeUSec; /* microsecond part */ + /* This will call exit() if strdup() fails. */ progname = get_progname(argv[0]); *** *** 915,920 --- 922,933 */ StartupPID = StartupDataBase(); + /* +* Get start up time +*/ + StartTimeSec = GetCurrentAbsoluteTimeUsec(&StartTimeUSec); + StartTime = AbsoluteTimeUsecToTimestampTz(StartTimeSec, StartTimeUSec); + status = ServerLoop(); /* *** *** 3669,3674 --- 3682,3688 write_inheritable_socket(¶m->pgStatPipe1, pgStatPipe[1], childPid); param->PostmasterPid = PostmasterPid; + param->StartTime = StartTime; #ifdef WIN32 param->PostmasterHandle = PostmasterHandle; *** *** 3871,3876 --- 3885,3891 read_inheritable_socket(&pgStatPipe[1], ¶m->pgStatPipe1); PostmasterPid = param->PostmasterPid; + StartTime = param->StartTime; #ifdef WIN32 PostmasterHandle = param->PostmasterHandle; *** ./src/backend/tcop/postgres.c.orig 2005-01-20 18:39:18.0 -0200 --- ./src/backend/tcop/postgres.c 2005-01-20 18:15:07.0 -0200 *** *** 144,149 --- 144,152 #endif /* TCOP_DONTUSENEWLINE */ + /* Backend startup time */ + TimestampTz StartTime; + /* *decls for routines only used in this file * *** *** 2217, --- 2220,2228 sigjmp_buf local_sigjmp_buf; volatile bool send_rfq = true; + AbsoluteTimeStartTimeSec; /* integer part */ + int StartTimeUSec; /* microsecond part */ + #define PendingConfigOption(name,val) \ (guc_names = lappend(guc_names, pstrdup(name)), \ guc_values = lappend(guc_values, pstrdup(val))) *** *** 2896,2901 --- 2902,2916 send_rfq = true;/* initially, or after error */ /* +* Get stand-alone backend startup time +*/
Re: [PATCHES] uptime function to postmaster
This has been saved for the 8.1 release: http:/momjian.postgresql.org/cgi-bin/pgpatches2 --- Euler Taveira de Oliveira wrote: > Hi, > > > > > > This patch just implement a function that returns the uptime of > > the > > > > postmaster. > > > > > > Doesn't work in EXEC_BACKEND case. > > > > > Sorry. I'll fix this. > > > Patch updated. I improved the function output too. > > Comments? > > > = > Euler Taveira de Oliveira > euler[at]yahoo_com_br > > __ > Converse com seus amigos em tempo real com o Yahoo! Messenger > http://br.download.yahoo.com/messenger/ Content-Description: uptime3.diff [ Attachment, skipping... ] > > ---(end of broadcast)--- > TIP 4: Don't 'kill -9' the postmaster -- Bruce Momjian| http://candle.pha.pa.us pgman@candle.pha.pa.us | (610) 359-1001 + If your life is a hard drive, | 13 Roberts Road + Christ can be your backup.| Newtown Square, Pennsylvania 19073 ---(end of broadcast)--- TIP 8: explain analyze is your friend
Re: [PATCHES] uptime function to postmaster
Hi Alvaro, > > > Patch updated. I improved the function output too. > > Is it really a good idea to have a new file for only one function? > We regularly see complaints about not wanting to do basic source code > cleanup because not wanting to disrupt CVS history ... > I couldn't see a good place to put it. Any suggestion? = Euler Taveira de Oliveira euler[at]yahoo_com_br ___ Yahoo! Acesso Grátis - Instale o discador do Yahoo! agora. http://br.acesso.yahoo.com/ - Internet rápida e grátis ---(end of broadcast)--- TIP 6: Have you searched our list archives? http://archives.postgresql.org
Re: [PATCHES] uptime function to postmaster
On Wed, Dec 22, 2004 at 12:57:34AM -0300, Euler Taveira de Oliveira wrote: Hi, > Patch updated. I improved the function output too. Is it really a good idea to have a new file for only one function? We regularly see complaints about not wanting to do basic source code cleanup because not wanting to disrupt CVS history ... -- Alvaro Herrera (<[EMAIL PROTECTED]>) "El hombre nunca sabe de lo que es capaz hasta que lo intenta" (C. Dickens) ---(end of broadcast)--- TIP 8: explain analyze is your friend
Re: [PATCHES] uptime function to postmaster
Hi, > > > This patch just implement a function that returns the uptime of > the > > > postmaster. > > > > Doesn't work in EXEC_BACKEND case. > > > Sorry. I'll fix this. > Patch updated. I improved the function output too. Comments? = Euler Taveira de Oliveira euler[at]yahoo_com_br __ Converse com seus amigos em tempo real com o Yahoo! Messenger http://br.download.yahoo.com/messenger/ uptime3.diff Description: uptime3.diff ---(end of broadcast)--- TIP 4: Don't 'kill -9' the postmaster
Re: [PATCHES] uptime function to postmaster
Hi Tom, > > This patch just implement a function that returns the uptime of the > > postmaster. > > Doesn't work in EXEC_BACKEND case. > Sorry. I'll fix this. = Euler Taveira de Oliveira euler[at]yahoo_com_br ___ Yahoo! Acesso Grátis - Instale o discador do Yahoo! agora. http://br.acesso.yahoo.com/ - Internet rápida e grátis ---(end of broadcast)--- TIP 1: subscribe and unsubscribe commands go to [EMAIL PROTECTED]
Re: [PATCHES] uptime function to postmaster
Euler Taveira de Oliveira <[EMAIL PROTECTED]> writes: > This patch just implement a function that returns the uptime of the > postmaster. Doesn't work in EXEC_BACKEND case. regards, tom lane ---(end of broadcast)--- TIP 1: subscribe and unsubscribe commands go to [EMAIL PROTECTED]