Re: [PATCHES] Small patch to replace 'idle' by 'trans' if transactionis still open

From: Christof Petig <christof(at)petig-baender(dot)de>
To: PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Cc: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>, The Hermit Hacker <scrappy(at)hub(dot)org>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-patches(at)postgresql(dot)org
Subject: Re: [PATCHES] Small patch to replace 'idle' by 'trans' if transactionis still open
Date: 2001-01-22 07:51:33
Message-ID: 3A6BE685.E9DA378A@petig-baender.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

If you don't know what is all about read the bottom (I'll requote my posting)

Bruce Momjian wrote:

> > On Fri, 19 Jan 2001, Bruce Momjian wrote:
> > > Seems we decided against this. Sorry.
> >
> > Huh? from reading Tom's response, sounds like it would be something
> > useful? I know I find having as much information about state in the ps
> > listing helps me alot, and knowing if its 'idle' vs 'idle (in
> > transaction)' provides at lesat more detailed information then just 'idle'
>
> > Who was the 'we' in the above decision? Tom seemed in favor of it, I know
> > I'm in favor of it .. and you are not in favor of it ...
>
> There must have been some discussion about it. I don't see it in the
> code, and I remember it was rejected for some reason. Check the archives.

The thing which comes most close to a rejection was the 'I can't decide' mail
by you (answered by Tom). The conclusion sounded like 'since we're not clear on
this subject we won't touch this, yet'. And there was some unsettled discussion
about the best wording to show in 'ps'.

'trans' seemed too short (and too much unlike 'idle') (as indicated by Bruce)
and 'idle (open transaction)' might give difficulties on platforms which limit
the length of the string (as indicated by Tom)

I'll CC Hackers (where this discussion belongs)

Christof

---------------------

Quoting:

Subject: Re: [PATCHES] Small patch to replace 'idle' by 'trans' if transaction
is still open
Date: Mon, 09 Oct 2000 22:46:56 -0400
From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>

Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> I can't decide if this is of general use. My inclination is that
> someone in a transaction sitting a prompt should still show as idle.

The idea seemed good to me, although I didn't look at the code to see
if the implementation was any good ;-). I know we've frequently had
questions on the lists where it was interesting to know if any
transactions were being held open --- and right now there's no easy
way to tell.

regards, tom lane

--------------

Subject: Small patch to replace 'idle' by 'trans' if transaction is still open
Date: Tue, 03 Oct 2000 21:28:36 +0200
From: Christof Petig <christof(dot)petig(at)wtal(dot)de>

If you are looking for programs which tend to hold longstanding
transactions, this micro patch might be handy.
Whether it is of general use is debatable. It will replace 'idle' by
'trans' if the backend is idle but a transaction is pending.

Simply use ps to view the backend's status.

Christof

--- src/backend/commands/async.c~ Sun May 14 05:18:35 2000
+++ src/backend/commands/async.c Tue Oct 3 10:31:54 2000
@@ -818,7 +818,7 @@
*/
pq_flush();

- PS_SET_STATUS("idle");
+ PS_SET_STATUS(IsTransactionBlock()?"trans":"idle");
TPRINTF(TRACE_NOTIFY, "ProcessIncomingNotify: done");
}

--- src/backend/tcop/postgres.c~ Thu Aug 31 09:18:57 2000
+++ src/backend/tcop/postgres.c Tue Oct 3 10:32:23 2000
@@ -1496,7 +1496,7 @@

for (;;)
{
- PS_SET_STATUS("idle");
+ PS_SET_STATUS(IsTransactionBlock()?"trans":"idle");

/* ----------------
* (1) tell the frontend we're ready for a new query.

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Magnus Hagander 2001-01-22 08:45:24 FW: Postgresql on win32
Previous Message Bruce Momjian 2001-01-22 06:35:28 Re: Need a debugging tip or two

Browse pgsql-patches by date

  From Date Subject
Next Message The Hermit Hacker 2001-01-22 13:00:49 Re: [PATCHES] Small patch to replace 'idle' by 'trans' if transactionis still open
Previous Message Robert B. Easter 2001-01-22 06:26:57 Re: docs: syntax.sgml patch