Re: Fix for broken JDBC's getColumn() (take 2)

From: "Vianen, Jeroen van" <jeroen(dot)van(dot)vianen(at)satama(dot)com>
To: pgsql-jdbc(at)postgresql(dot)org, pgsql-patches(at)postgresql(dot)org
Subject: Re: Fix for broken JDBC's getColumn() (take 2)
Date: 2001-09-17 07:59:07
Message-ID: F9926D32A30ED511B8E30050044AB52E101489@AMS010
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-jdbc pgsql-patches

Hi,

What's the status of this patch I sent a week ago?

Thanks,

Jeroen

> -----Original Message-----
> From: Jeroen van Vianen [mailto:jeroen(dot)van(dot)vianen(at)satama(dot)com]
> Sent: Monday, September 10, 2001 20:01
> To: pgsql-jdbc(at)postgresql(dot)org; pgsql-patches(at)postgresql(dot)org
> Subject: [JDBC] Fix for broken JDBC's getColumn() (take 2)
>
>
> Hi,
>
> Per the recent discussion there's been some code changes in JDBC's
> DatabaseMetaData.getColumn(). I proposed a patch that would
> change the
> number of queries to find out all columns in a table from 2 *
> N + 1 to 1 (N
> being the number of columns reported) by using some outer
> joins. I also
> fixed the fact that getColumns() only returned columns that
> had a default
> defined. OTOH, I did not use to change the code required for
> obtaining a
> column's remarks (by using col_description() for 7.2 and
> requested by Tom
> Lane).
>
> Finally, I have found a way to get all the column details in
> a single query
> *and* use col_description() for 7.2 servers. A patch is attached. It
> overrules René Pijlman's fix for this that was committed just
> today, but
> still used N + 1 queries (sorry René ;-) )
>
> I also fixed the return values for TABLE_CAT and TABLE_SCHEM
> from "" to
> null, to be more standard compliant (and requested in René's
> mail found at
> http://fts.postgresql.org/db/mw/msg.html?mid=1034253).
>
> As always, the JDBC1 version has not been tested as I have no JDK 1.1
> available.
>
> Please review,
>
>
> Jeroen
>
>

Browse pgsql-jdbc by date

  From Date Subject
Next Message Justin Clift 2001-09-17 10:25:56 Re: newbie question: loading the jdbc driver & connecting to my
Previous Message Russ McBride 2001-09-17 07:49:31 newbie question: loading the jdbc driver & connecting to my db

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2001-09-17 15:56:08 Re: isNullable()
Previous Message Barry Lind 2001-09-17 04:38:03 Re: [PATCHES] isNullable()