Skip site navigation (1) Skip section navigation (2)

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: Fix for broken JDBC's getColumn() (take 2)
Date: 2001-09-27 14:05:22
Message-ID: F9926D32A30ED511B8E30050044AB52E10149A@ams010.satama.com (view raw or flat)
Thread:
Lists: pgsql-jdbcpgsql-patches
Hi,

What's the status of this patch I sent two weeks ago? I got no reply last
week for the same question.

My original mail can be found at
http://fts.postgresql.org/db/mw/msg.html?mid=1034457

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

Responses

pgsql-patches by date

Next:From: Karel ZakDate: 2001-09-27 15:26:56
Subject: Re: Problem with setlocale (found in libecpg) [accessing a memory location after freeing it]
Previous:From: Christof PetigDate: 2001-09-27 10:17:31
Subject: Re: [PATCHES] Problem with setlocale (found in libecpg) [accessing a

pgsql-jdbc by date

Next:From: Matt FairDate: 2001-09-27 14:51:23
Subject: Re: blobs in a client/server environment
Previous:From: Nick FankhauserDate: 2001-09-27 13:26:04
Subject: Re: Re. conection error

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group