Re: [COMMITTERS] pgsql: Provide much better wait information in pg_stat_activity.

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: Joel Jacobson <joel(at)trustly(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [COMMITTERS] pgsql: Provide much better wait information in pg_stat_activity.
Date: 2016-03-11 02:36:14
Message-ID: CA+TgmoZQ7xawzh2K+DX7wRYjaiKHwQ61bRrysKJqFeTrNX6Ltw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

On Thu, Mar 10, 2016 at 8:31 PM, Joel Jacobson <joel(at)trustly(dot)com> wrote:
> This is an excellent feature, thanks!
> But can we please keep the old boolean waiting column?
> I see no reason to break backward-compatibility. Or maybe I'm missing something.

Well, this was discussed. If we keep the Boolean "waiting" column, then either:

1. We make it true only for heavyweight lock waits, and false for
other kinds of waits. That's pretty strange.
2. We make it true for all kinds of waits that we now know how to
report. That still breaks compatibility.

I do understand that changing this is backward-incompatible and a lot
of people are going to have to update their monitoring tools. But I
think that's the best alternative. If we choose option #1, we're
going to be saddled with a weird backward-compatibility column
forever, and ten years from now we'll be explaining that even if
waiting = false you might still be waiting depending on the value of
some other column. If we choose option #2, it won't be
backward-compatible and some people's queries will still break, just
less obviously. Neither of those things seems very appealing.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Robert Haas 2016-03-11 02:38:23 pgsql: Simplify GetLockNameFromTagType.
Previous Message Peter Eisentraut 2016-03-11 02:36:11 Re: pgsql: Checkpoint sorting and balancing.

Browse pgsql-hackers by date

  From Date Subject
Next Message Peter Geoghegan 2016-03-11 02:38:28 Re: Fix for OpenSSL error queue bug
Previous Message Peter Eisentraut 2016-03-11 02:35:45 Re: pgsql: Provide much better wait information in pg_stat_activity.