]> granicus.if.org Git - postgresql/commit
On Fri, 07 Sep 2001 01:34:46 -0400, Tom Lane wrote:
authorBruce Momjian <bruce@momjian.us>
Mon, 10 Sep 2001 14:55:08 +0000 (14:55 +0000)
committerBruce Momjian <bruce@momjian.us>
Mon, 10 Sep 2001 14:55:08 +0000 (14:55 +0000)
commit6b50f9af333a2795b82dd3331df55c7e703f1bea
tree58e172d984387cb20793866c2cc17590ea06f322
parentc69bb04acc13e0c1aa4e0b5e7910f6850d71173b
On Fri, 07 Sep 2001 01:34:46 -0400, Tom Lane wrote:
>there is still an unpatched reference to pg_description in
>getColumns(), in both jdbc1 and jdbc2.

This was introduced by Jeroen's patch (see
http://fts.postgresql.org/db/mw/msg.html?mid=1032468). Attached
is a patch that returns getColumns() to using "select
obj_description()" instead of direct access to pg_description,
as per the request by Tom.

I've incorporated Jeroen's fix to left outer join with
pg_attrdef instead of inner join, so getColumns() also returns
columns without a default value.

I have, however, not included Jeroen's attempt to combine
multiple queries into one huge multi-join query for better
performance, because:
1) I don't know how to do that using obj_description() instead
of direct access to pg_description
2) I don't think a performance improvement (if any) in this
method is very important

Because of the outer join, getColumns() will only work with a
backend >= 7.1. Since the conditional coding for 7.1/7.2 and
jdbc1/jdbc2 is already giving me headaches I didn't pursue a
pre-7.1 solution.

Regards,
Ren? Pijlman <rene@lab.applinet.nl>
src/interfaces/jdbc/org/postgresql/jdbc1/DatabaseMetaData.java
src/interfaces/jdbc/org/postgresql/jdbc2/DatabaseMetaData.java