Somebody broke \d on indexes

On 2013-11-14 09:52:11 -0500, Tom Lane wrote:

In HEAD:

regression=# \d tenk1_thous_tenthous
ERROR: column i.indisidentity does not exist
LINE 4: i.indisidentity,
^

That's me. At some point indisidentity was renamed to indisreplident.

Patch attached (also renaming a variable that didn't cause problems but
wasn't named consistently anymore).

Shouldn't we have at least one \d of an index in the regression tests
somewhere? Not that that excuses stupid mitakes, but it'd be helpful
nonetheless.

Greetings,

Andres Freund

--
Andres Freund http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services

Andres Freund <andres@2ndquadrant.com> writes:

On 2013-11-14 09:52:11 -0500, Tom Lane wrote:

In HEAD:

regression=# \d tenk1_thous_tenthous
ERROR: column i.indisidentity does not exist
LINE 4: i.indisidentity,
^

That's me. At some point indisidentity was renamed to indisreplident.

Patch attached (also renaming a variable that didn't cause problems but
wasn't named consistently anymore).

Ah, thanks, will commit.

Shouldn't we have at least one \d of an index in the regression tests
somewhere? Not that that excuses stupid mitakes, but it'd be helpful
nonetheless.

Seems like a good idea, will add one of those too.

regards, tom lane

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers