Re: BUG #12746: Backward compatibility is broken 9.2: pg_stat_activity.procpid renamed to pid

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: nad2000(at)gmail(dot)com
Cc: pgsql-bugs(at)postgresql(dot)org
Subject: Re: BUG #12746: Backward compatibility is broken 9.2: pg_stat_activity.procpid renamed to pid
Date: 2015-02-08 23:01:27
Message-ID: 9032.1423436487@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

nad2000(at)gmail(dot)com writes:
> I run an app that should support both 9.0 and 9.4. I found that at 9.2 the
> pg_stat_activity column porcpid was renamed to pid. It broke the app. To
> solve it I had to introduce conditional querying for different versions.
> I don't think it's right time to do any aesthetically motivated changes at
> this stage.

The commit log entry about that provides a bit of context:

commit 4f42b546fd87a80be30c53a0f2c897acb826ad52
Author: Magnus Hagander <magnus(at)hagander(dot)net>
Date: Thu Jan 19 14:19:20 2012 +0100

Separate state from query string in pg_stat_activity

This separates the state (running/idle/idleintransaction etc) into
it's own field ("state"), and leaves the query field containing just
query text.

The query text will now mean "current query" when a query is running
and "last query" in other states. Accordingly,the field has been
renamed from current_query to query.

Since backwards compatibility was broken anyway to make that, the procpid
field has also been renamed to pid - along with the same field in
pg_stat_replication for consistency.

Scott Mead and Magnus Hagander, review work from Greg Smith

Basically the point being that you're going to need some version
conditionality *anyway* in pretty much any app using this view.

In any case, as Marko noted, you're about three years too late to complain
about this. We're certainly not changing back now.

regards, tom lane

In response to

Browse pgsql-bugs by date

  From Date Subject
Next Message deanelliotdeloach 2015-02-09 00:02:33 BUG #12747: WARNING: unrecognized node type: 701
Previous Message Andres Freund 2015-02-08 23:00:08 Re: BUG #12746: Backward compatibility is broken 9.2: pg_stat_activity.procpid renamed to pid