Re: Small patch to replace 'idle' by 'trans' if transaction is still open

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Philip Warner <pjw(at)rhyme(dot)com(dot)au>
Cc: Christof Petig <christof(dot)petig(at)wtal(dot)de>, Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>, pgsql-patches(at)postgresql(dot)org
Subject: Re: Small patch to replace 'idle' by 'trans' if transaction is still open
Date: 2000-10-11 15:15:54
Message-ID: 22906.971277354@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Philip Warner <pjw(at)rhyme(dot)com(dot)au> writes:
> The idea was to indicate if the process is likely to be causing a locking
> problem - but now I think about it, a more useful pice of information would
> be if any tables/rows/whatever are locked in such a way that another user
> might be prevented from doing an update. Is this possible?

Don't see how you could convey a useful amount of info in the space
available. There are way too many levels of way too many locks that
might or might not conflict with someone else. (Even a read-only
transaction grabs read locks, so it's not like the presence or absence
of any lock is sufficient info to be useful.)

There was another thread recently about adding a SHOW command or some
such to dump out the state of the lock manager's table in a readable
form. I think that'd be a more useful thing to work on than trying
to cram one or two bits' worth of info into the ps display.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Stephenson 2000-10-11 15:52:07 JDBC Large ResultSet problem + BadTimeStamp Patch
Previous Message Philip Warner 2000-10-11 15:14:58 Re: Small patch to replace 'idle' by 'trans' if transaction is still open

Browse pgsql-patches by date

  From Date Subject
Next Message Philip Warner 2000-10-11 16:12:47 Re: Small patch to replace 'idle' by 'trans' if transaction is still open
Previous Message Philip Warner 2000-10-11 15:14:58 Re: Small patch to replace 'idle' by 'trans' if transaction is still open